Conversation
|
💩 Code linting failed, use |
Summary
|
parfenovma
left a comment
parfenovma
left a comment
There was a problem hiding this comment.
Кмк лучше отдельную ручку под бота сделать)
| 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?