Skip to content

Добавил approved_by#114

Merged
parfenovma merged 5 commits intomainfrom
pivo
Apr 12, 2025
Merged

Добавил approved_by#114
parfenovma merged 5 commits intomainfrom
pivo

Conversation

@MikhailPI1
Copy link
Copy Markdown
Contributor

@MikhailPI1 MikhailPI1 commented Apr 9, 2025

Изменения

Добавил логгирование пользователя, а именно колонка approved_by.

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

Смотри изменения.

Check-List

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 9, 2025

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

@MikhailPI1 MikhailPI1 self-assigned this Apr 9, 2025
@MikhailPI1 MikhailPI1 linked an issue Apr 9, 2025 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 9, 2025

Summary

Tests Skipped Failures Errors Time
51 0 💤 0 ❌ 0 🔥 8.715s ⏱️

@MikhailPI1 MikhailPI1 requested a review from zipperman1 April 9, 2025 19:37
@zipperman1 zipperman1 requested a review from Zimovchik April 9, 2025 19:38
Comment thread migrations/versions/8206267a15ff_approved_by.py Outdated
Comment thread migrations/versions/8206267a15ff_approved_by.py Outdated
Comment thread migrations/versions/8206267a15ff_approved_by.py Outdated
Comment thread migrations/versions/8206267a15ff_approved_by.py Outdated
Comment thread rating_api/routes/comment.py Outdated
Comment thread rating_api/routes/comment.py
@Zimovchik
Copy link
Copy Markdown
Member

а еще перед отправкой кода пропиши в консоль black . и isort . Это просто форматирование кода

@github-actions
Copy link
Copy Markdown

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

@parfenovma parfenovma merged commit ff1ab65 into main Apr 12, 2025
2 checks passed
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.

Логгировать автора аппрува

4 participants