Popup new bal already exist

fufeck commented 11 months ago

CONTEXT

Il y a des confusions lors de la création des BALs, certains ne savent pas que c'est déjà géré pour eux ou qu'ils ont déjà une BAL.

FONCTIONNALITE

Ajout d'une popup d'avertissement a la création d'une BAL si une révision existe deja

RESULTAT

Si une BAL a déjà été publié via mes-adresses

Si une BAL a déjà été publié via le moissonneur

Si une BAL a déjà été publié via l'api-depot

PR

MaGOs92 commented 10 months ago

Test :

J'ai une pop up vide quand il n'y a pas d'id legacy sur le client :
Il faudrait virer le premier if (client.id) dans alert-published-bal ligne 30

MaGOs92 commented 10 months ago

Tests :

J'ai une erreur lors du call de BasesLocalesService.searchBaseLocale dans la fonction checkOtherBALs. Apparement l'API attend un flag 0 ou 1 plutôt que true / false pour le param deleted

MaGOs92 commented 10 months ago

Les méthodes "onSubmit" et "checkOtherBALs" des composants upload-form et create-form sont quasiment identiques. C'était pas possible de les déclarer dans la page new et de les passer en props pour éviter de dupliquer le code?

fufeck commented 10 months ago

Tests :

J'ai une erreur lors du call de BasesLocalesService.searchBaseLocale dans la fonction checkOtherBALs. Apparement l'API attend un flag 0 ou 1 plutôt que true / false pour le param deleted

Il y a une PR sur mes-adresses-api pour ca https://github.com/BaseAdresseNationale/mes-adresses-api/pull/423/files

BaseAdresseNationale / mes-adresses