Skip to content

chore(API Entreprise): Mise à jour des données via l'API Recherche Entreprise#1678

Merged
SebastienReuiller merged 12 commits intomasterfrom
sreuiller/api-recherche-entreprises
Mar 31, 2025
Merged

chore(API Entreprise): Mise à jour des données via l'API Recherche Entreprise#1678
SebastienReuiller merged 12 commits intomasterfrom
sreuiller/api-recherche-entreprises

Conversation

@SebastienReuiller
Copy link
Copy Markdown
Contributor

Quoi ?

Mise à jour des champs "api_entreprise_*" avec l'API de Recherche Entreprise

Pourquoi ?

API Entreprise a été mise à jour et nous n'avons pas les accès à la nouvelle version. La PR #1599 ne peut donc pas être mise en production.
L'API Recherche Entreprise est quant à elle ouverte.

Comment ?

Appel en GET à l'API de Recherche Entreprise

Autre

L'ajout de from sentry_sdk.crons import monitor permettra de suivre la bonne exécution de la tache planifiée.

Prochaine étape :

  • mettre à jour aussi les données anciennes, pas uniquement que les structures pour lesquelles le champs n'a jamais été renseigné.
  • gérer le quota d'appel à l'API

Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/utils/apis/api_recherche_entreprises.py Outdated
Comment thread lemarche/utils/apis/api_recherche_entreprises.py Outdated
Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/utils/apis/api_recherche_entreprises.py Outdated
Comment thread lemarche/siaes/tests/test_commands.py
Comment thread lemarche/utils/apis/api_recherche_entreprises.py
Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/siaes/tests/test_commands.py Outdated
Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/siaes/management/commands/update_api_entreprise_fields.py Outdated
Comment thread lemarche/utils/apis/api_recherche_entreprises.py Outdated
Comment thread lemarche/siaes/tests/test_commands.py Outdated
@SebastienReuiller SebastienReuiller force-pushed the sreuiller/api-recherche-entreprises branch from 466fa52 to d7e0f08 Compare March 27, 2025 17:42
Copy link
Copy Markdown
Contributor

@Guilouf Guilouf left a comment

Choose a reason for hiding this comment

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

Nickel !

@SebastienReuiller SebastienReuiller merged commit c4c4654 into master Mar 31, 2025
8 checks passed
@SebastienReuiller SebastienReuiller deleted the sreuiller/api-recherche-entreprises branch March 31, 2025 09:38
@sentry
Copy link
Copy Markdown

sentry Bot commented Apr 1, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ KeyboardInterrupt lemarche.siaes.management.commands.update_api_e... View Issue

Did you find this useful? React with a 👍 or 👎

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.

2 participants