Skip to content

Comments

feat/qg-290: реализованы уведомления о заполнении расписания#292

Merged
d-r-q merged 22 commits intomasterfrom
qg-290/push-notifications
Nov 5, 2025
Merged

feat/qg-290: реализованы уведомления о заполнении расписания#292
d-r-q merged 22 commits intomasterfrom
qg-290/push-notifications

Conversation

@d-r-q
Copy link
Member

@d-r-q d-r-q commented Oct 16, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Oct 16, 2025

Unit Test Results

  74 files    74 suites   1m 26s ⏱️
249 tests 249 ✔️ 0 💤 0
258 runs  258 ✔️ 0 💤 0

Results for commit 2c93851.

♻️ This comment has been updated with latest results.

d-r-q added 18 commits October 16, 2025 13:48
…жения и репозиторий переведён на работу с ней
…ена на пакетный режим.

Условно - пакетно достаются подписки, а сами пуши так и отправляются поштучно, т.к. пакетного режима нет (как минимум в используемой либе)
Новое АПИ работает через лямбды и should-ы и должно решить проблемы старого АПИ.

Со старым было две проблемы:

1. Заведение PageObject-ов/Component-ов требовало слишком много церемоний
2. Использование матчеров имело ещё две проблемы:
2.1 при фейле сложного дерева матчеров в стектрейсе не было указания на зафейлившуюся проверку, а из сообщений это зачастую не всегда очевидно. Соответственно периодически приходилось проводить детективные расследования
2.2 Матчеры тоже требуют много церемоний
За счёт того, что в метод открытия страницы добавлено ожидание фактического открытия страницы - при наличии ожидается смена title-а страницы, а если его нет - появление тоггла сайдбара.
Т.к. делает тоже самое и задепрекейчен
…ment shouldHave ComponentMatcher

Т.к. второй лучше соответсвует смыслу, где использовался первый
…ификаций и отображение корректных дня и времени оповещения о заполнении расписания
@d-r-q d-r-q enabled auto-merge November 5, 2025 11:19
@d-r-q d-r-q merged commit d074d5b into master Nov 5, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant