BaseAdresseNationale / mes-adresses

Éditeur de Bases Adresses Locales
https://mes-adresses.data.gouv.fr
10 stars 3 forks source link

Refacto status publication #855

Closed fufeck closed 10 months ago

fufeck commented 12 months ago

PROBLEME

Le system de publication est un peu compliqué a comprendre et est perturbant lorsqu'il y a une mise a jour programmé et pas d'habilitation.

FONCTIONNALITEE

RESULTAT

On prend une BAL qui est a jour mais sans habilitation

Capture d’écran 2024-01-15 à 10 47 52

On fait une modification

Capture d’écran 2024-01-15 à 10 48 06

Au bout de 30s le status devient En attente d'habilitation

Capture d’écran 2024-01-15 à 10 48 56

Après s'être authentifier (2h après) le status redevient a jour

Capture d’écran 2024-01-15 à 10 47 08
fufeck commented 10 months ago

Code and tests LGTM, Remarque :

  • Refresh du statut quand l'habilitation a expiré : il reste "à jour" alors qu'il devrait passer à "en attente d'habilitation" après une modification. Il faut attendre que le CRON passe côté back et rafraichir la page pour voir le statut "en attente d'habilitation". D'un point de vu utilisateur, ça me parait pas très clair. Si on ne peut pas mettre à jour le statut à la volée, je pense qu'il faut mieux afficher "en attente d'habilitation" avant modif

C'est corrigé, maintenant dès qu'il y a une modification alors que l'habilitation a expiré est Synchronisation en cours pendant 30s et ensuite en attente d'habilitation

MaGOs92 commented 10 months ago

Juste une remarque, il faudrait vraiment repasser à du rebase plutôt que du merge parce que 54 commits ça rend la review très fastidieuse et je ne sais pas exactement ce qui est lié à la feature et ce qui est déjà mergé