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

feat(poc): add new routes on v1 #384

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hlecuyer
Copy link
Contributor

@hlecuyer hlecuyer commented Feb 13, 2025

Descriptif des changements apportés :

  • Refactorisation des namespaces : création des dossiers v1 et v0 sous src/data_inclusion/api/.
  • Migration des dossiers analytics, inclusion_data et inclusion_schema.
  • Création du dossier v0 dans les tests.
  • Réorganisation des modèles Pydantic : déplacement des modèles du schéma actuel vers un sous-package dédié : src/data_inclusion/api/v0/inclusion_schema/legacy.
  • Mise en place d'un namespace via les routers FastAPI (cf. src/data_inclusion/api/app.py).
  • Ajout de routes pour servir des fichiers contenant quelques services/structures au format du package data_inclusion_schema (src/data_inclusion/api/v1/inclusion_data/routes.py).
  • La partie SQLAlchemy/DB a été mise de côté pour l’instant.

Questions :

  • Quels éléments de v0 peut-on factoriser avec v1 ?

    • src/data_inclusion/api/v0/analytics/models.py → Très bon candidat, on souhaite garder une structure commune entre les versions.
    • src/data_inclusion/api/v0/inclusion_data/commands.py → Plus discutable.
    • Y a-t-il d'autres éléments que j’aurais manqués ?
  • D'autres fichiers avec des dépendances sont-ils a dupliqués dans v0 et `v1 ?

    • src/data_inclusion/api/utils/soliguide.py ?
    • src/data_inclusion/api/decoupage_administratif/commands.py ?

@hlecuyer hlecuyer self-assigned this Feb 13, 2025
@hlecuyer hlecuyer requested a review from vmttn as a code owner February 13, 2025 15:08
@hlecuyer hlecuyer marked this pull request as draft February 13, 2025 15:09
@hlecuyer hlecuyer force-pushed the hlecuyer/poc/start-versioning-on-api branch from 5456528 to 5c6fc23 Compare February 17, 2025 09:04
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.

1 participant