Skip to content

Document source#60

Merged
Temmmmmo merged 12 commits intomainfrom
document_source
Jul 26, 2023
Merged

Document source#60
Temmmmmo merged 12 commits intomainfrom
document_source

Conversation

@Temmmmmo
Copy link
Copy Markdown
Member

Изменения

При отправке файла на печать теперь также указывается источник отправки.

Реализации

Добавлен параметр source в POST-ручке /file, а также добавлена строчка в БД.

Детали

В JSON-модель добавлен string-параметр source, а также в БД File добавлено поле source.

Check-List

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

@Temmmmmo Temmmmmo linked an issue Jul 26, 2023 that may be closed by this pull request
5 tasks
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 26, 2023

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/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.py1301985%49, 55, 57, 129–130, 171, 179–180, 182, 185–186, 191–192, 202–205, 247, 256
   qrprint.py844744%36–45, 50–56, 59–66, 69–78, 81, 84–87, 95–98, 106–111
   user.py51492%66, 113–115
print_service/utils
   __init__.py58395%44, 86–87
TOTAL62612780% 

Summary

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

Wudext
Wudext previously requested changes Jul 26, 2023
Copy link
Copy Markdown

@Wudext Wudext left a comment

Choose a reason for hiding this comment

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

У тебя при мердже в main может возникнуть проблема с тем, что у существующих записей нет параметра source (который у тебя not null) и ты уронишь тест (или прод). Алембик в этом плане тупенткий, поэтому либо в моделе БД в срочке где добавлен source пиши default='что-то', либо прописывай руками в миграции задание нужных значений (что ой как не весело).

Comment thread migrations/versions/a68c6bb2972c_add_document_source.py
@github-actions
Copy link
Copy Markdown

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

@Temmmmmo Temmmmmo self-assigned this Jul 26, 2023
Comment thread migrations/versions/a68c6bb2972c_add_document_source.py Outdated
@Temmmmmo Temmmmmo requested a review from Wudext July 26, 2023 21:44
@Temmmmmo Temmmmmo dismissed Wudext’s stale review July 26, 2023 21:45

Approved by grigoriev-semyon

@Temmmmmo Temmmmmo merged commit 2109ae7 into main Jul 26, 2023
@Temmmmmo Temmmmmo deleted the document_source branch July 26, 2023 21:46
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.

Добавить source при отправке файла на печать в print-api

3 participants