Develop alert bot #123
Develop alert bot #123
Conversation
|
💩 Code linting failed, use |
Summary
|
| user_id: int | None = None, | ||
| order_by: list[Literal["create_ts"]] = Query(default=[]), | ||
| unreviewed: bool = False, | ||
| review_mode: Literal["all", "pending"] = "all", # вот ключ |
There was a problem hiding this comment.
А чем отличается от unreviewed?
There was a problem hiding this comment.
не стал переписывать unreviewd т.к возможно какаято логика завязана. Новый возвращает только pending без DISSMISED и еще чемто отлтичается
There was a problem hiding this comment.
лучше тогда, если уж так начал сделать, сделать явно запрос по любому статусу DISMISSED, PENDING, REVIEW
Если они не проставлены, то возвращаются все
Но, чтобы можно было вернуть, чисто по статусу одному, нужны скоупы
There was a problem hiding this comment.
короче очень костыльно, если твоим способом
лучше так, как я предложил, но нужно это будет согласовать с фронтом
rating_api/routes/comment.py
Outdated
|
|
||
| comments = query.all() | ||
| if not comments: | ||
| return CommentGetAll(polls=[], limit=limit, offset=offset, total=0) |
There was a problem hiding this comment.
В чем отличие если нет комментариев возвращать [] либо 404?
There was a problem hiding this comment.
На что влияет и где почитать
There was a problem hiding this comment.
в целом ок 200 возвращать
3 participants
Изменения
Добавил функционал в логику GET comment
Детали реализации
Check-List
blackиisortдля Back-End илиPrettierдля Front-End?