Immatriculation principale

etalab / rncs_worker_api_entreprise

API pour récupérer et mettre à disposition les données du Répertoire National du Commerce et des Société

https://entreprise.data.gouv.fr

MIT License

23 stars 13 forks source link

Immatriculation principale #148

Closed Haelle closed 5 years ago

Haelle commented 5 years ago

Passage de 1 à 3 messages d'erreurs :

aucune immatriculation trouvée
aucune immatriculation principale trouvée
erreur dans l'immatriculation principale

Je ne suis pas super fan de l'opération ; c'est pas très lisible car il y a 5 "sorties" possibles (le cas idéal, le cas où on prend l'immatriculation la plus récente et les 3 messages d'erreurs). On peut peut-être faire mieux avec des opérations nestées mais j'aime bien l'idée d'avoir tout au même endroit.

Haelle commented 5 years ago

maj faites :

blank? au lieu de nil?
fail_fast rattrapé dans l'opération parente !

:+1: si ok pour merge