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

(BSR)[API] feat: Rationalisation des blueprints de l'API publique #12928

Merged

Conversation

tcoudray-pass
Copy link

@tcoudray-pass tcoudray-pass commented Jun 25, 2024

…ings_blueprint`

But de la pull request

Refacto pour supprimer les blueprints intermédiaires :

  • individual_bookings_blueprint
  • individual_offers_blueprint
  • collective_offers_blueprint
  • v1_bookings_blueprint
  • v1_offers_blueprint
  • v2_prefixed_public_api
  • public_v1_blueprint

Ils rajoutaient de la complexité non nécessaire. Cette PR permettra également de plus facilement retrouver un endpoint avec un Ctrl + F basé sur l'URL (auparavant cela était impossible car l'url était tronçonnée en plusieurs morceau /public puis /{resource} puis /v1)

Vérifications

  • J'ai écrit les tests nécessaires
  • J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
  • J'ai ajouté des screenshots pour d'éventuels changements graphiques

@tcoudray-pass tcoudray-pass changed the title (BSR)[API] feat: remove unecessary v1_*_blueprint & `individal_book… (BSR)[API] feat: Rationalisation des blueprints de l'API publique Jun 25, 2024
@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/BSR-simplify-individual-offers-blueprint branch 2 times, most recently from 2c7af37 to 5b3b1c9 Compare June 25, 2024 15:13
@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/BSR-simplify-individual-offers-blueprint branch from 5b3b1c9 to 8396d73 Compare June 26, 2024 08:07
Copy link
Contributor

@jeremieb-pass jeremieb-pass left a comment

Choose a reason for hiding this comment

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

Merci ! Ça devrait effectivement simplifier les choses.

Est-ce que ça ne vaut pas le coup de renommer aussi le fichier blueprints en blueprint (au singulier) ?

Et peut-être aussi le blueprint lui-même en public_api tout court ? Histoire d'éviter les blueprint.public_api_blueprint qui sont un peu répétitifs.

@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/BSR-simplify-individual-offers-blueprint branch from 8396d73 to 352b422 Compare June 26, 2024 13:43
@tcoudray-pass tcoudray-pass force-pushed the tcoudray-pass/BSR-simplify-individual-offers-blueprint branch from 352b422 to 37ff04c Compare June 26, 2024 13:50
@tcoudray-pass tcoudray-pass merged commit 66cccf7 into master Jun 26, 2024
22 checks passed
@tcoudray-pass tcoudray-pass deleted the tcoudray-pass/BSR-simplify-individual-offers-blueprint branch June 26, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants