Get comment reaction by tishanovartem · Pull Request #156 · profcomff/rating-api

tishanovartem · 2025-11-12T21:46:35Z

Что сделано:

В ручке GET /comment/{uuid} теперь возвращаются флаги is_liked и is_disliked, которые показывают поставил ли текущий пользователь лайк/дизлайк на комментарий.
Реализован гибридный метод has_reaction() в модели Comment для эффективной проверки реакций пользователь.

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

Check-List

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

github-actions · 2025-11-12T21:47:07Z

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

github-actions · 2025-11-12T21:48:43Z

Coverage Report

File	Stmts	Miss	Cover	Missing
rating_api
__main__.py	4	4	0%	1–7
exceptions.py	43	7	84%	35–37, 48–50, 58
rating_api/models
base.py	64	4	94%	24–27
db.py	164	18	89%	115, 117, 119, 121, 132, 171, 189, 199, 213, 222–224, 257, 271–283
rating_api/routes
base.py	16	1	94%	41
comment.py	144	35	76%	64, 82, 131–132, 153–162, 172, 236–237, 239–240, 248–253, 262, 269–271, 295, 308, 335–346, 377
exc_handlers.py	32	3	91%	36, 43, 50
lecturer.py	109	9	92%	70–71, 81, 87–88, 217, 225, 243, 249
rating_api/schemas
base.py	12	4	67%	6–9
models.py	155	3	98%	189, 191, 201
rating_api/utils
mark.py	8	8	0%	1–20
TOTAL	785	96	88%

Summary

Tests	Skipped	Failures	Errors	Time
88	0 💤	0 ❌	0 🔥	25.487s ⏱️

Zimovchik

нужно и в ручке get comments тоже это сделать

Zimovchik · 2025-12-13T07:24:15Z

rating_api/models/db.py

+    @classmethod
+    def reactions_for_comments(cls, user_id: int, session, comments):
+        if not user_id or not comments:
+            return {}
+        comments_uuid = [c.uuid for c in comments]
+        reactions = (
+            session.query(CommentReaction)
+            .filter(CommentReaction.user_id == user_id, CommentReaction.comment_uuid.in_(comments_uuid))
+            .all()
+        )
+        return {r.comment_uuid: r.reaction for r in reactions}


ты можешь сделать это методами алхимии, групнув сджойненые таблички например и посмотрев 1 0 или -1

github-actions · 2026-03-05T14:17:09Z

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

petrCher · 2026-03-07T20:43:13Z

Сейчас такая проблема: ты добавил новую схему CommentGetWithLike, наследующуяся от CommentGet. Сперва кажется что решение красивое, но появляется масса проблем при новое схеме. Ты не учел, например, что в ручках получения лекторов тоже есть вывод комментариев, который остался со старой логикой CommentGet. Также пришлось добавлять новые схемы, что неудобно. Не понял, кстати, зачем ты закомменченные схемы добавил? Если на будущее, то об этом лучше писать в чат бэкенда как идея на будущее.
Предлагаю сейчас просто добавить два новых булевых поля is_liked и is_disliked добавить в исходную схему CommentGet, тогда проблем не будет

petrCher · 2026-03-07T20:14:46Z

rating_api/routes/comment.py


-@comment.get("", response_model=Union[CommentGetAll, CommentGetAllWithAllInfo, CommentGetAllWithStatus])
+@comment.get(
+    "", response_model=Union[CommentGetAll, CommentGetAllWithLike, CommentGetAllWithAllInfo, CommentGetAllWithStatus]


здесь надо привезти это к одной строке обратно

ну и плюс убрать лишнюю схему

petrCher · 2026-03-07T20:24:26Z

rating_api/routes/comment.py


     `asc_order` -Если передано true, сортировать в порядке возрастания. Иначе - в порядке убывания
    """
+    user = {"id": 101, "session_scopes": []}  # тестовый user_id  # пустые скоупы = обычный пользователь


:)
если это для тестов у тебя было, рекомендую убрать))

petrCher · 2026-03-07T20:32:26Z

rating_api/schemas/models.py



+"""
+class CommentGetWithLikeAndStatus(CommentGetWithLike):


зачем?
ты хочешь добавить ревью на лайки/дизлайки?) это точно не надо делать будет

petrCher · 2026-03-07T20:36:01Z

rating_api/schemas/models.py



+"""
+class CommentGetWithAllInfoAndLike(CommentGetWithLike):


убрать

petrCher · 2026-03-07T20:36:10Z

rating_api/schemas/models.py

    total: int


+"""


убрать

petrCher · 2026-03-07T20:36:42Z

rating_api/schemas/models.py

    total: int


+"""


убрать

petrCher · 2026-03-07T20:37:35Z

rating_api/schemas/models.py

    total: int


+class CommentGetAllWithLike(Base):


чувствуешь как усложнилась логика из-за добавления схемы CommentGetWithLike?

petrCher · 2026-03-07T20:46:04Z

tests/test_routes/test_comment.py

+    if reaction_data:
+        user_id, reaction_type = reaction_data
+        reaction = CommentReaction(user_id=user_id, comment_uuid=comment.uuid, reaction=reaction_type)
+        dbsession.add(reaction)


все что связано с обращением к базе данных должно быть вынесено в conftest.py
Если есть вопросы как это делать напиши мне, помогу. А так можешь чекнуть аналогичные соседние тесты или в рентал апи зайти и посмотреть как там

petrCher · 2026-03-07T20:55:28Z

rating_api/routes/comment.py


    result.total = len(result.comments)
-    result.comments = [comment_validator.model_validate(comment) for comment in result.comments]
+    comments_with_like = []


здесь пока логику не проверил, чекну как только исправишь что я написал

github-actions · 2026-03-09T19:53:27Z

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

tishanovartem requested review from MikhailPI1 and Zimovchik November 12, 2025 21:46

tishanovartem self-assigned this Nov 12, 2025

tishanovartem linked an issue Nov 12, 2025 that may be closed by this pull request

в get_comment выводить для юзера, лайкнул ли он этот коммент #154

Open

Zimovchik requested changes Nov 14, 2025

View reviewed changes

Zimovchik requested changes Dec 13, 2025

View reviewed changes

tishanovartem requested review from petrCher and removed request for MikhailPI1 March 5, 2026 18:36

petrCher requested changes Mar 7, 2026

View reviewed changes

tishanovartem added 7 commits March 9, 2026 22:37

Добавление флагов is_liked и is_disliked

9f4eb62

Исправление для пустого комментария в тестах

8c5a15e

Небольшое исправление README

c5358ba

Применение black и isort

28f1edd

Добавление реакций для get_comments

d558235

Исправление получения реакция для нескольких комментариев

0388ac0

Изменения в get_comment и get_comments

533a554

tishanovartem force-pushed the get_comment_reaction branch from 3858a9d to 533a554 Compare March 9, 2026 19:52

Conversation

tishanovartem commented Nov 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Что сделано:

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

Check-List

Uh oh!

github-actions bot commented Nov 12, 2025

Uh oh!

github-actions bot commented Nov 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

Zimovchik left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Mar 5, 2026

Uh oh!

petrCher commented Mar 7, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Mar 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

tishanovartem commented Nov 12, 2025 •

edited

Loading

github-actions bot commented Nov 12, 2025 •

edited

Loading