Skip to content

API: Simplifier l'API SIAE - #6804

Merged
tonial merged 1 commit into
masterfrom
alaurent/api
Sep 5, 2025
Merged

tonial merged 1 commit into
masterfrom
alaurent/api

Conversation

@tonial

@tonial tonial commented Sep 5, 2025 •

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Suppression de l'action retrieve qui n'est jamais utilisée (l'ID nécessaire n'est pas exposé par la liste) et donc les données renvoyées sont identiques à celles de la liste.

Suppression de la description écrite à le main du format de réponse au profit de celui autogénéré par la documentation.

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

Remove retrieve method that doesn't work since we don't send the id used
to retreive a company

Clean the documentation (the response format is automatically generated,
no need to write it manually)
@tonial
tonial requested a review from xavfernandez September 5, 2025 11:47
@tonial tonial self-assigned this Sep 5, 2025
@tonial tonial added the modifié Modifié dans le changelog. label Sep 5, 2025
@tonial
tonial added this pull request to the merge queue Sep 5, 2025
Merged via the queue into master with commit f8fb44d Sep 5, 2025
15 checks passed
@tonial
tonial deleted the alaurent/api branch September 5, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants