Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Получение информации о статусе комментов пользователя #91

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

zipperman1
Copy link
Contributor

@zipperman1 zipperman1 commented Feb 10, 2025

Изменения

возможность запросить неаппрувнутые комменты если id совпадает с указанным user_id

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

см выше

Check-List

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

Copy link

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

Copy link

github-actions bot commented Feb 10, 2025

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
rating_api
   __main__.py440%1–7
   exceptions.py34779%35–37, 48–50, 58
rating_api/models
   base.py55591%24–27, 76
   db.py73297%61, 74
rating_api/routes
   comment.py1012278%34, 38, 56, 74, 109–111, 132–141, 184, 194–202, 209, 233, 249
   exc_handlers.py23387%33, 40, 47
   lecturer.py841483%132–155, 171, 179, 197, 203
rating_api/schemas
   base.py12467%6–9
TOTAL5056188% 

Summary

Tests Skipped Failures Errors Time
40 0 💤 0 ❌ 0 🔥 7.588s ⏱️

Copy link
Member

@Temmmmmo Temmmmmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нужно возвращать модель, где будет также поле review_status

@zipperman1 zipperman1 changed the title add userid check Получение информации о статусе его комментов Feb 22, 2025
@zipperman1 zipperman1 changed the title Получение информации о статусе его комментов Получение информации о статусе комментов пользователя Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants