betagouv / ma-cantine

Accompagner les acteurs restau co dans la loi EGalim
https://ma-cantine.beta.gouv.fr/
MIT License
14 stars 9 forks source link

Échec import : état de resolution #3700

Open hfroot opened 6 months ago

hfroot commented 6 months ago

Dans le modèle, ajouter :

state = choix de "en cours"/"resolu"/"abandonné" successful_import_source = text

écrire des tests et le code pour faire en sort que quand un import marche, on marque les anciens échecs du même utilisateur et même type d'import comme résolu, et que l'import_source pour l'import est ajouté au successful_import_source sur tous les échecs.

Dans l'admin, view liste des échecs, ajouter une colonne d'état, et des filtres, genre CanteenAdmin.télédéclarée

abandonné j'imagine pour les cas où un utilisateur s'est trompé et utilisé le mauvais type d'import, alors l'échec pourrais jamais être resolu automatiquement. Mais je suis pas convaincu de ce nom ou même la logique, preneur de suggestions.


Dans une deuxième PR on pourrait ajouter une automatisation qui supprime les fichiers des echecs resolus (après une semaine ?).

qloridant commented 6 months ago

Comment on s'assure que le fichier d'import est le même (modulo les erreurs) ?

hfroot commented 6 months ago

Le fichier sera forcement pas le même (car besoin de le modifier pour qu'il passe). Je me suis dit qu'un utilisateur va pas travailler sur plusieurs imports de même type en même temps. En tout cas les echecs resolus seronts toujours trouvables, et si on utilise un couche de temps (une semaine/un mois etc) avant supprimer le fichier, on pourrait toujours trouver un echec et le marquer comme non resolu si l'utilisateur nous contacte ?

J'ai p-e proposé une solution sans bien définir le pb, alors je le fais maintenant :