Cette PR propose plusieurs corrections de différentes erreurs détectées à l'import :
Bug lors de l'enchainement des mises à jours qui échouait lors de l'import de la dernière MAJ de la file
Suppression des guillemets et espaces blancs autour de la donnée avant de la sauvegarder en base (pour éviter des numéro de siren enregistrés en base avec des quotes "\"123456789\""
Pas de création de doublons en base quand des doublons sont transmis dans les fichiers.
Avec ceci vient un petit peu de refactor qui va bien et l'ajout d'un peu de doc dans le README.
Comment faire la PR
Commit par commit
Ce qu'il reste à faire
Comme je le disais il y a eu du refactor, ce n'est pas encore parfait. Pour rendre le code plus clair il y a maintenant un peu de duplication, rien de bien méchant (je pense notamment au fichier app/concepts/tribunal_commerce/daily_update_unit/operation/import_transmission.rb) , ça doit pouvoir partir avec un coup de meta-prog. Mais c'est pas urgent et ce sera pour plus tard. J'ai noté en TODO ce que je voulais faire pour améliorer ça.
Contenu de la PR
Cette PR propose plusieurs corrections de différentes erreurs détectées à l'import :
"\"123456789\""Avec ceci vient un petit peu de refactor qui va bien et l'ajout d'un peu de doc dans le README.
Comment faire la PR
Commit par commit
Ce qu'il reste à faire
Comme je le disais il y a eu du refactor, ce n'est pas encore parfait. Pour rendre le code plus clair il y a maintenant un peu de duplication, rien de bien méchant (je pense notamment au fichier app/concepts/tribunal_commerce/daily_update_unit/operation/import_transmission.rb) , ça doit pouvoir partir avec un coup de meta-prog. Mais c'est pas urgent et ce sera pour plus tard. J'ai noté en TODO ce que je voulais faire pour améliorer ça.