[E-64] Update the current state of the web application #104
Conversation
MrAmfix
commented
Feb 20, 2025
MrAmfix
commented
- Interaction with the database has been moved to async mode
- Updated schemas
- Updated_models suggested
- Added logger
- Other small fixes
There was a problem hiding this comment.
Помимо предыдущих комментариев еще пара моментов:
- Все измененные файлы нужно покрыть докстрингами. То, что было до этой задачи тоже.
- Почему возникла необходимость такой чистки в requirements?
- Изменения в любом случае будем сливать после обновления CI/CD. Пока не проходят пайплайны мы не можем их пропустить.
| @@ -9,6 +9,9 @@ __pycache__/ | |||
| .vscode | |||
| **/__pycache__ | |||
| **/test* | |||
| migration | |||
| alembic.ini | |||
| test_folder | |||
There was a problem hiding this comment.
А для чего нужна эта директория?
There was a problem hiding this comment.
test_folder там просто файлы для проверки работы некоторых функций или посылок запросов
alembic_helper.txt
Outdated
There was a problem hiding this comment.
По хорошему нужно этот файл добавить в README веб-приложения в раздел инструкции по запуску.
There was a problem hiding this comment.
Сейчас пытаюсь в E-90 сделать скрипт, чтобы миграции автоматически сразу проводились, тогда этот файл не понадобится
docker-compose-database.yml
Outdated
| services: | ||
| edulytica_db: | ||
| container_name: "EdulyticaDatabase" | ||
| image: postgres:14.1-alpine |
There was a problem hiding this comment.
Может повыше версию PG возьмем, 15-16?
src/edulytica_api/app.py
Outdated
| # load_dotenv() | ||
| # models.Base.metadata.create_all(bind=engine) |
There was a problem hiding this comment.
Почему остался закомментированный код?
src/edulytica_api/crud/factory.py
Outdated
There was a problem hiding this comment.
Новые и старые методы нужно покрыть докстрингами.
There was a problem hiding this comment.
Почему сейчас осталось без реализации?
|
Я в requirements ничего не чистил, я просто добавил свои библиотеки и перезаписал файл, из-за этого некоторые библиотеки поменяли вид, например было "pyjwt", стало "PyJWT==2.10.1". |
