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

Ignorer les données associées à des dossiers inconnus #150

Closed brindu closed 4 years ago

brindu commented 4 years ago

Contenu de la PR

Ne pas tenir comptes des représentants, établissements ou observations, qu'ils soient créés ou mis à jour, lorsque l'identifiant de la PM ou PP est inconnu en base de données. La ligne est ignorée, le cas est loggé et l'import continue.

Voir #39.

hugolepetit commented 4 years ago

Bon pour moi, ne connaissant pas trop la façon dont tu logges je veux bien a l'occaz que tu me dises comment le lien se fait entre ctx warning et nos fichiers de logs

brindu commented 4 years ago

Le message de warning est catch ici et simplement loggé, pour une erreur l'import est stoppé.

brindu commented 4 years ago

Comme je le disais dans le slack, il n'y a aucune erreur de ce type en réalité : le problème venait d'un défaut dans les fichiers téléchargés... Je clos donc cette PR, pas besoin d'être trop laxiste avant l'heure, on verra le jour où on rencontrera vraiment cette situation.