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

Добавление софт делитов в принтер #87

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

Conversation

gitfresnel
Copy link
Contributor

@gitfresnel gitfresnel commented Oct 23, 2024

Изменения

  1. Добавил в таблицу union_member поле is_deleted
  2. Изменил поведение ручек (get/is_union_member, post/is_union_member, post/file, post/file/{pin}, patch/file/{pin}) в соответствии с новым полем
  3. Добавил тесты для ручек
  4. Изменил декларативный класс (метод query не возвращает поля с софт делитами)

Check-List

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

@gitfresnel gitfresnel linked an issue Oct 23, 2024 that may be closed by this pull request
Copy link

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

Copy link

github-actions bot commented Oct 23, 2024

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
print_service
   __main__.py440%1–7
   base.py12467%6–9
   exceptions.py52787%21, 40, 45, 60, 65, 70, 77
print_service/models
   base.py531179%24–27, 56–57, 70–75
print_service/routes
   admin.py492647%28–29, 32–36, 39–43, 50–56, 63–69
   auth.py440%1–8
   exc_handlers.py59985%34, 58, 70, 80, 124, 136, 144, 154, 166
   file.py1261985%50, 56, 58, 131–132, 180, 189–190, 192, 195–196, 201–202, 212–215, 261, 270
   qrprint.py1127137%39–48, 53–59, 62–70, 73–82, 86–107, 110, 113–116, 124–127, 135–148
   user.py50198%64
print_service/utils
   __init__.py57395%44, 87–88
TOTAL69715977% 

Summary

Tests Skipped Failures Errors Time
27 1 💤 0 ❌ 0 🔥 1.234s ⏱️

…member_.py

Удаление старого файла миграции
…р-1' into 29-добавить-софт-делиты-в-принтер-1

# Conflicts:
#	migrations/versions/120e06710249_add_is_deleted_field_to_unionmember_.py
print_service/routes/file.py Outdated Show resolved Hide resolved
print_service/routes/user.py Outdated Show resolved Hide resolved
print_service/routes/user.py Outdated Show resolved Hide resolved
tests/test_routes/conftest.py Show resolved Hide resolved
print_service/routes/exc_handlers.py Outdated Show resolved Hide resolved
print_service/models/__init__.py Show resolved Hide resolved
print_service/models/base.py Outdated Show resolved Hide resolved
print_service/routes/user.py Show resolved Hide resolved
print_service/routes/file.py Show resolved Hide resolved
print_service/routes/user.py Outdated Show resolved Hide resolved
tests/test_routes/test_user.py Show resolved Hide resolved
Copy link

github-actions bot commented Nov 5, 2024

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

print_service/models/base.py Outdated Show resolved Hide resolved
print_service/routes/user.py Outdated Show resolved Hide resolved
…р-1' into 29-добавить-софт-делиты-в-принтер-1

# Conflicts:
#	migrations/versions/808f98fc21f5_add_is_deleted_field_to_unionmember.py
…р-1' into 29-добавить-софт-делиты-в-принтер-1

# Conflicts:
#	migrations/versions/c29b6ffbfed4_add_is_deleted_field_to_unionmember.py
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