Skip to content

patch archive handler#26

Open
rifcrat wants to merge 2 commits into
mainfrom
patch_id_archive_handler
Open

patch archive handler#26
rifcrat wants to merge 2 commits into
mainfrom
patch_id_archive_handler

Conversation

@rifcrat

@rifcrat rifcrat commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Изменения

Добавил ручку PATCH/notification/{id}/archive

Детали реализации

В файле schemas\models.py добавил новую схему для ответа
В файле routes\notes.py написал саму ручку
В файле utils\services.py добавил логику для ручки

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@rifcrat rifcrat requested a review from petrCher July 7, 2026 22:44
@rifcrat rifcrat self-assigned this Jul 7, 2026
@rifcrat rifcrat linked an issue Jul 7, 2026 that may be closed by this pull request

@petrCher petrCher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

А почему не тестил код?
Надо делать еще небольшой user story что пользователь архивировал модалку и потом делает get запрос, а там та модалка со статусом active

Comment thread modal_backend/routes/notes.py Outdated
Comment thread modal_backend/utils/services.py Outdated
@petrCher

petrCher commented Jul 8, 2026

Copy link
Copy Markdown
Member

Как сделаешь изменения скинь на ревью, планирую потом дать тебе доработать ручку
План на потом:

  1. добавить возможность возврата из архива в этой же ручке (например, нужно для подготовления уведомлений и постов заранее)
  2. поменять название пути ручки на что-то универсальное (с archive на status например)
  3. в логике возврата проверять на возможность возобновления работы модалки (например, проблема может быть с уже несуществующими группами или с истекшим временем)
  4. аргументы в параметрах ручки сделать не только для айди но и для статуса (см https://github.com/profcomff/rating-api/blob/main/rating_api/routes/comment.py#L207-L209 )

@petrCher

This comment was marked as resolved.

@rifcrat rifcrat force-pushed the patch_id_archive_handler branch from a2381df to 68f174a Compare July 10, 2026 21:30
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.

Добавить архивацию модалки

2 participants