Skip to content

Get note's comment for user - #42

Merged
petrCher merged 4 commits into
mainfrom
get_notes_for_user_id
Sep 5, 2026
Merged

Get note's comment for user#42
petrCher merged 4 commits into
mainfrom
get_notes_for_user_id

Conversation

@ArtemBratyashin

@ArtemBratyashin ArtemBratyashin commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Изменения

Добавил ручку, которая получает список комментариев для 3-ех видов модалок.

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

Можно выбрать лимит и отступ. Протестировал в сваггере.

Check-List

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

@ArtemBratyashin ArtemBratyashin changed the title Get note's omment for user Get note's comment for user Aug 28, 2026
@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
modal_backend
   __main__.py440%1–6
   exceptions.py20195%37
modal_backend/models
   base.py62789%22, 25–28, 57, 87
modal_backend/routes
   exc_handlers.py17194%38
   notes.py61198%251
modal_backend/schemas
   base.py12467%6–9
TOTAL5401897% 

Summary

Tests Skipped Failures Errors Time
85 0 💤 0 ❌ 0 🔥 8.308s ⏱️

@ArtemBratyashin ArtemBratyashin linked an issue Aug 28, 2026 that may be closed by this pull request
@ArtemBratyashin ArtemBratyashin self-assigned this Aug 28, 2026

@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.

все норм, мерджу, единственное на коммент обрати внимание)

id: int,
limit: int = Query(10, ge=0, description="Лимит записей"),
offset: int = Query(0, ge=0, description="Смещение записей на N+offset, где N - первая запись"),
user=Depends(UnionAuth(scopes=["modal.note.read"])),

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.

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

@petrCher
petrCher merged commit 6416293 into main Sep 5, 2026
2 checks passed
@petrCher
petrCher deleted the get_notes_for_user_id branch September 5, 2026 06:48
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.

Ручка note response

2 participants