Skip to content

Conversation

@hgfejcwoefGleb
Copy link

Изменения

Добавил софт делиты. Ориентировался на код в пр #87

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

1.Изменил структуру модуля models, чтобы был похож на rating api и тд
2.Добавил в таблицу union_member поле is_deleted
3.Изменил поведение ручек file/send/, file/upload_file, update_file_options в соответствии с новой логикой базовой модели
4.Поменял название поля класс UserCreate с username на surname, так как возникали ошибки в тестах
5.Изменил тесты в соответствии с новой структурой классов
6.Добавил новые исключения PrintAPIError, ObjectNotFound, AlreadyExists

Check-List

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

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

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

@github-actions
Copy link

github-actions bot commented Oct 9, 2025

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
print_service
   __main__.py440%1–7
   base.py12467%6–9
   exceptions.py62985%16, 31, 50, 55, 70, 75, 80, 92, 97
print_service/models
   base.py621871%21, 53, 56, 58–59, 66–77, 84
   db.py61198%71
print_service/routes
   admin.py492647%28–29, 32–36, 39–43, 50–56, 63–69
   auth.py440%1–8
   exc_handlers.py701283%37, 61, 73, 83, 127, 139, 147, 157, 169, 217, 237, 247
   file.py1292184%47, 53, 55, 133–134, 142–143, 188, 196–197, 199, 202–203, 208–209, 219–222, 264, 273
   qrprint.py1127137%39–48, 53–59, 62–70, 73–82, 86–107, 110, 113–116, 124–127, 135–148
   user.py48296%66, 114
print_service/utils
   __init__.py59985%43, 85–86, 91–100
TOTAL73718175% 

Summary

Tests Skipped Failures Errors Time
23 1 💤 3 ❌ 0 🔥 1.246s ⏱️

@hgfejcwoefGleb hgfejcwoefGleb self-assigned this Oct 9, 2025
@VladislavVoskoboinik
Copy link

по братски убери алембиковские комменты в миграции

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.

2 participants