Пользовательская ручка view #38

[Aiz0r]

Изменения

Добавлена ручка POST /user/{id}/view, она фиксирует реальный показ модалки пользователю. Показ засчитывается в таблице note_view: при первом показе создаётся запись, при повторных увеличивается счётчик показов. Если модалки не существует - ошибка 404, если модалка не активна - ошибка 403

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

NoteViewService.mark_view (utils/services.py) - основная логика
Тесты в tests/test_routes/test_user.py: первый показ, повторный показ (инкремент), несуществующая модалка (404), неактивная модалка (403)

Check-List

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

[github-actions]

💩 Code linting failed, use black and isort to fix it.

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
modal_backend
__main__.py	4	4	0%	1–6
exceptions.py	20	1	95%	37
modal_backend/models
base.py	62	7	89%	22, 25–28, 57, 87
modal_backend/routes
exc_handlers.py	17	1	94%	38
notes.py	61	1	98%	251
modal_backend/schemas
base.py	12	4	67%	6–9
modal_backend/utils
user_logic.py	22	2	91%	22, 26
TOTAL	576	20	97%

Summary

Tests	Skipped	Failures	Errors	Time
89	0 💤	0 ❌	0 🔥	8.219s ⏱️

[Aiz0r]

пофиксил формат, вроде теперь норм

[petrCher]

тесты еще не смотрел, сперва надо поправить мои комменты

плюс еще дополнение небольшое, возможно непонятно:
visit_count в другой таблице (ты с ней здесь не работаешь) просто считает вообще все заходы в сервис пользователя
shown_count считает сколько раз показали именно эту модалку
last_visit_number это копия visit_count на момент последнего показа

[petrCher]

@Aiz0r если будут вопросы, то лучше прямо сюда пиши тегнув меня

@Georgon тоже рекомендую посмотреть и если есть, что написать делай ревью прямо в коде со своими комментами

[Aiz0r]

Вроде все пофиксил, жду ревью

[petrCher]

@Aiz0r когда исправляешь мой коммент, надо его резолвить, чтобы я видел что он действительно исправлен
также есть комменты по поводу линтинга, которые не изменились, там где кавычки и переносы на другую строку

[petrCher]

когда доделаешь попроси @Georgon отревьюить, потом я ревью сделаю финальный и замерджим
Гоша как раз сейчас вроде освободился от срочных дел)

[petrCher]

@Aiz0r заливай изменения скорее и мердж будет)
пора закрывать таску

[Aiz0r]

Все сделал, можно мерджить @petrCher