etalab / entreprise.api.gouv.fr

Site publique d'API Entreprise
https://entreprise.api.gouv.fr
7 stars 7 forks source link

Ajout d'information dans les docs INSEE suite au Webinaire #108

Closed DorineLam closed 3 years ago

DorineLam commented 3 years ago

Que fait cette PR ?

Elle propose d'ajouter dans la doc des éléments de contenus donnés par l'INSEE lors du Webinaire du 30 mars 15h. Webinaire coorganisé par l'API Entreprise et l'INSEE

Pourquoi fait-on ça ? Contexte, PR, issue liée ?

Pour compléter la documentation

DorineLam commented 3 years ago

J'ai repéré deux informations intéressantes parmi les questions des utlisateurs et les réponses de l'INSEE :

skelz0r commented 3 years ago

Pour les sirets nous n'acceptons en paramètre que les sirets respectant le format de Luhn SAUF pour la poste (siret sous le format /356000000/) : est-ce que tu as des exemples de sirets ne respectant pas cette formule du coup ? Si tu me fournis un exemple valide je suis quasiment sûr qu'on renverra un 422, et qu'il faudra itérer.

DorineLam commented 3 years ago

Super intéressant, et bien d'après l'INSEE : "les entreprises de réseau (EDF, Orange. …)" n'auraient pas un SIRET de ce format Luhn "en raison de contraintes techniques". C'est ce que j'ai lu dans le CR envoyé du Webinaire, je n'en sais pas plus que ça

skelz0r commented 3 years ago

@Haelle invocation!

skelz0r commented 3 years ago

J'ai checké rapidement j'ai trouvé aucun siret pour EDF.

DorineLam commented 3 years ago

J'ai checké rapidement j'ai trouvé aucun siret pour EDF.

@skelz0r, du coup ? Qu'est-ce que cela signifie à ton sens ?

skelz0r commented 3 years ago

Ne pas avoir trouvé d'exemple n'implique pas qu'ils n'existent pas, j'ai scrollé et testé genre 30 sirets, tous valide vis-à-vis de la formule de Luhn.

Sur wikipedia il ne parle que du cas de La Poste (déjà géré dans siade), donc je peux pas t'en dire plus.

DorineLam commented 3 years ago

@Haelle et @skelz0r je merge cette PR. Elle permet au moins d'ajouter la question de la fraicheur.