Skip to content

Bogdan Borutenko @upir6tihii9#3

Open
bborutenko wants to merge 9 commits intoTevenixLevelUps:mainfrom
bborutenko:main
Open

Bogdan Borutenko @upir6tihii9#3
bborutenko wants to merge 9 commits intoTevenixLevelUps:mainfrom
bborutenko:main

Conversation

@bborutenko
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Contributor

@TimoxGagarin TimoxGagarin left a comment

Choose a reason for hiding this comment

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

  1. Добавь .gitignore. Добавь туда venv, pycache. Не забудь добавить requirements.txt со списком зависимостей.
  2. Добавь разделение по файлам. Экспертов, услуги, заказы храни отдельно друг от друга. Схемы храни отдельно от эндпоинтов. Почитай про APIRouter в FastAPI. У тебя должна быть примерно такая файловая структура.:
    schemas
    |--experts.py
    |--orders.py
    |--services.py
    api
    |--experts.py
    |--orders.py
    |--services.py
    main.py
  3. Крайне не рекомендую назначать 1 эндпоинт для работы со всеми сущностями. Это сильно снижает масштабируемость сервиса.
  4. По тз в моделях должно быть еще и поле id
  5. По тз у некоторых моделей должны быть еще эндпоинты для patch
  6. Обычно для 1 сущности в pydantic создаются 3 модели (Create, Update, Display).
  7. При создании сущности обычно данные передаются как Body, а не Query
  8. Эндпоинт db не совсем уместный

Мб чет пропустил, но исправь сначала это. Потом кину новый фидбэк.

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