Sweetbread
1 Followers · 0 Following
Repositories
17
 Projects Packages Public Activity Starred Repositories
1
Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init

Слишком длинная строка

Sweetbread suggested changes for Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init
  1. print'ы вместо логов
Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init

Слишком длинное определение. Лучше заменить на

Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init

Слишком длинное определение

Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init

Зачем?

Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 14:10:31 +00:00
feat/db-init

Можно ли заменить на User(*row)?

Sweetbread commented on pull request Hackaton/Backend#3 2025-10-26 13:52:36 +00:00
feat/db-init

А в чём смысл обрабатывать ошибку, которая всё равно уронит процесс? Если она неожиданная, то и обрабатывать…

Sweetbread pushed to dev at Hackaton/Backend 2025-10-26 13:24:06 +00:00
90f1a6245c feat(log): use loguru
532266c98e refactor(auth): remove /api/auth/* handlers to api/auth.py
f99c0e8148 refactor(user): rename to toJson back
2a7c74286c feat(auth): add data validation
920c51b4f0 refactor(user): change class name to uppercase
Compare 6 commits »
Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 17:36:18 +00:00
Implement login API, closes #1

Такого файла нет

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 15:43:19 +00:00
Implement login API, closes #1

Я вижу что это. Где используется?

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 15:43:05 +00:00
Implement login API, closes #1

Я вижу что это. Где используется?

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 15:31:47 +00:00
Implement login API, closes #1

Так а зачем PR отправил?

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 15:31:27 +00:00
Implement login API, closes #1

ТЗ

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:21:26 +00:00
Implement login API, closes #1

jsonify для корректных заголовков ответа и отсутствия костылей

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1
  1. Зачем f-строки? Если переменные и так строки, то просто вставляй их
Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1

Выглядит, как что-то недоработанное

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1
  1. Отдели имя и аннотацию пробелами (email: str, ...)
Sweetbread suggested changes for Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1
  1. Не по ТЗ
Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1

Где используется?

Sweetbread commented on pull request Hackaton/Backend#2 2025-10-25 10:14:11 +00:00
Implement login API, closes #1

Попробуй просто по порядку перечислять аргументы, а то x=x так себе выглядит