Skip to content

[4138.1] Notification Delivery Settings - #111

Merged
niqzart merged 6 commits into
stagingfrom
feat/notification-delivery-settings
Aug 12, 2026
Merged

[4138.1] Notification Delivery Settings#111
niqzart merged 6 commits into
stagingfrom
feat/notification-delivery-settings

Conversation

@niqzart

@niqzart niqzart commented Jul 7, 2026

Copy link
Copy Markdown
Member

No description provided.

@niqzart niqzart changed the title Notification Delivery Settings [???] Notification Delivery Settings Jul 7, 2026
@niqzart niqzart self-assigned this Jul 7, 2026
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch 5 times, most recently from 019c60b to dd20ba5 Compare July 7, 2026 02:45
@niqzart niqzart added the ci:covered If coverage checks should be run on the PR label Jul 7, 2026
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch 5 times, most recently from da18bee to 4fc9653 Compare July 7, 2026 03:27
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 4fc9653 to 1651107 Compare July 8, 2026 00:48
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch 5 times, most recently from 805920a to 5c4aee0 Compare July 11, 2026 01:08
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 5c4aee0 to ef3217a Compare July 11, 2026 01:31
@niqzart niqzart added the ci:deployable If deployment of the PR should be attempted label Jul 11, 2026
@niqzart
niqzart changed the base branch from feat/classroom-event-reminders to devops/manual-deloyment-everywhere July 11, 2026 01:33
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 2c9b3e7 to 6eeb9d5 Compare July 11, 2026 01:36
@niqzart niqzart removed the ci:deployable If deployment of the PR should be attempted label Jul 12, 2026
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch 2 times, most recently from bda6e7f to 0548088 Compare July 22, 2026 01:00
@niqzart niqzart changed the title [???] Notification Delivery Settings [4138.1] Notification Delivery Settings Jul 25, 2026
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 0548088 to d659cdd Compare July 25, 2026 02:46

@ByrDen ByrDen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Как ты в этой логике не запутался...
Чуть-чуть мелких правок нашёл

Comment thread app/notifications/routes/delivery_methods_rst.py
Comment thread app/notifications/models/delivery_methods_db.py
Comment thread app/notifications/models/delivery_methods_db.py
Comment thread app/notifications/models/user_contacts_db.py
Comment thread app/notifications/routes/delivery_methods_rst.py
Comment thread tests/notifications/functional/test_delivery_methods_int.py Outdated
Comment thread tests/notifications/service/senders/test_telegram_notification_sender.py Outdated
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from d659cdd to cf5e878 Compare July 27, 2026 10:14

@ByrDen ByrDen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вопросов больше нет :)

@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from cf5e878 to d1c0166 Compare August 5, 2026 16:18
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from d1c0166 to 6dee8a1 Compare August 8, 2026 13:00
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 6dee8a1 to 51e0fb7 Compare August 8, 2026 15:15
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from 51e0fb7 to f6bcf0b Compare August 11, 2026 20:39
Base automatically changed from devops/manual-deloyment-everywhere to staging August 12, 2026 01:58
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from f6bcf0b to b049415 Compare August 12, 2026 01:58
@niqzart
niqzart force-pushed the feat/notification-delivery-settings branch from b049415 to 9378e1c Compare August 12, 2026 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:covered If coverage checks should be run on the PR ci:migrated If migration checks should be run on the PR ci:trigger Use to trigger the development pipeline (re-run)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants