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

🐛 Affichage de l'adresse du siège dans les résultats de la recherche entreprise #2884

Merged
merged 5 commits into from
Feb 7, 2024

Conversation

bulteau
Copy link
Contributor

@bulteau bulteau commented Feb 3, 2024

fix #2883

Description de la PR

  • Ne plus utiliser directement firstMatchingEtablissement car cela ne correspond pas nécessairement au siège.
  • Récupération de l'établissement siège depuis le json de résultat retourné par l'api https://api.recherche-entreprises.fabrique.social.gouv.fr/ en parcourant le tableau des allMatchingEtablissements
  • Si aucun siège, affichage du premier établissement trouvé : firstMatchingEtablissement

Autres modifications

  • Renommer is_siege par etablissementSiege : l'api a évolué (cf commit sur le repo SocialGouv/recherche-entreprises)
  • L'api ne retourne pas toujours le bon code postal dans le tableau des établissements allMatchingEtablissements. La fonction fetchCommuneDetails a été adaptée pour ne pas faire de vérification de concordance entre le code postal et le code commune dans ce cas de figure. Vérification seulement utile pour la recherche d'une commune dans le cas d'un input utilisateur. Issue créé sur le repo de l'api socialgouv.

Copy link

netlify bot commented Feb 3, 2024

Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name Link
🔨 Latest commit 6ad010c
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/65be0bd5965cdf00085034e1

@bulteau bulteau changed the title 🐛 Affichage de l'adresse du siège dans les résultats de la recherche entreprise #2883 🐛 Affichage de l'adresse du siège dans les résultats de la recherche entreprise Feb 3, 2024
Copy link
Contributor

@johangirod johangirod left a comment

Choose a reason for hiding this comment

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

Merci pour la PR ! Effectivement, nous avons prévu de changer pour l'API recherche entreprise plutôt que celle du MTES, qui n'est plus maintenue.

@johangirod johangirod merged commit 84f5c6d into betagouv:master Feb 7, 2024
5 checks passed
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.

🐛 Affichage de l'adresse du siège dans les résultats de la recherche entreprise
2 participants