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

Changement siren vers siret dans le 'path' #1670

Merged
merged 4 commits into from
Sep 18, 2024

Conversation

DorineLam
Copy link
Contributor

No description provided.

@DorineLam
Copy link
Contributor Author

@skelz0r il y avait encore une mention siren dans le path du chemin yaml. J'avoue avoir du mal à comprendre à quoi sert ce chemin si dans le swagger c'était différent ? Peux-tu m'expliquer d'un point de vue tech ?

@skelz0r
Copy link
Member

skelz0r commented Sep 17, 2024

Parce que ce n'est pas le bon endpoint je pense, si le contexte est ce thread : https://mattermost.incubateur.net/betagouv/pl/xo8fr5k1itnqpb9p67q173f58c on parle de la DGFIP et non de BdF

@DorineLam
Copy link
Contributor Author

Parce que ce n'est pas le bon endpoint je pense, si le contexte est ce thread : https://mattermost.incubateur.net/betagouv/pl/xo8fr5k1itnqpb9p67q173f58c on parle de la DGFIP et non de BdF

Ok donc on vient de merger des changements dans la doc qui ne sont pas les bons.
@Charlottecho je corrige, c'était chiffres d'affaires qu'il fallait changer ?
Peux-tu reconfirmer ? En attendant je prépare cette PR pour correction

@Charlottecho
Copy link
Contributor

@skelz0r : pour info : le FS avait transmis une url de l'API 3 derniers bilans BDF , au départ. Erreur de sa part car il parlait de l'API CA . Sauf que dans cette API "3 derniers bilans de france " , l'erreur est la même ( siren dans doc métier et siret dans swagger ) ... On est en train de creuser cela avec @DorineLam .

@skelz0r
Copy link
Member

skelz0r commented Sep 18, 2024

@skelz0r : pour info : le FS avait transmis une url de l'API 3 derniers bilans BDF , au départ. Erreur de sa part car il parlait de l'API CA . Sauf que dans cette API "3 derniers bilans de france " , l'erreur est la même ( siren dans doc métier et siret dans swagger ) ... On est en train de creuser cela avec @DorineLam .

Si on parle du thread mattermost il n'y a aucune mention de la BdF.

En tout cas le swagger fait foi no matter what (si il est faux ça ne marche tout simplement pas).

@DorineLam
Copy link
Contributor Author

Pour conclure, on avait aussi une erreur dans banque de france, ou c'était marqué SIRET alors qu'il fallait un SIREN.

@Charlottecho tout à l'heure j'ai en plus eu un quiproquo en confondant le champ "paramètre d'appel" avec le recipient (siret de l'administration qui fait l'appel) ce qui m'a complètement embrouillée !

Je merge !

@DorineLam DorineLam merged commit 91b5d3b into develop Sep 18, 2024
9 checks passed
@DorineLam DorineLam deleted the API-3406-changement-siren-vers-siret branch September 18, 2024 13:53
@Charlottecho
Copy link
Contributor

Charlottecho commented Sep 19, 2024

@skelz0r il avait balancé le lien de l'API BDF au départ qu'il a modifié , d'ou son message "j'ai transmis le mauvais lien" 🙃 dans le thread mattermost . Mais on est tout bon . Cela a mis en lumière un autre problème dans la doc qu'il y avait concernant l'API BDF .
@DorineLam : les modifications n'ont pas été apportées ici . Je le fais ?

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.

4 participants