search

BaseAdresseNationale / adresse.data.gouv.fr

Le site officiel de l'Adresse
https://adresse.data.gouv.fr
MIT License
153 stars 31 forks source link

Refacto : Gestion des partenaires de la charte depuis BAL-Admin #1584

Closed MaGOs92 closed 1 year ago

MaGOs92 commented 1 year ago

Contexte

Afin de simplifier le process d'ajout des partenaires de la charte, cette PR permet de gérer les partenaires directement depuis BAL-Admin.

PR Bal-admin : https://github.com/BaseAdresseNationale/bal-admin/pull/28

MaGOs92 commented 1 year ago

Mince j'arrive pas à reproduire le bug de la page commune, c'est pas un problème de conf de ton api-geo?

MaGOs92 commented 1 year ago

Pour le problème d'envoi de mail j'ai fixé ça côté bal-admin avec un try catch.

MaGOs92 commented 1 year ago

Pour les logs d'erreurs dans la console et le problème de design, ça devrait être bon

fufeck commented 1 year ago

Mince j'arrive pas à reproduire le bug de la page commune, c'est pas un problème de conf de ton api-geo?

Alors après quelque recherche, j'ai remarqué que ces a cause des communes partenaire qui nont pas de codeDepartement Capture d’écran 2023-08-30 à 15 15 58

Or les 3 communes, Saclay que j'ai créé a la volée pour test nont pas de codeDepartement

MaGOs92 commented 1 year ago

J'ai plus le log de problème d'hydration depuis que j'ai ajouté styled-component dans la config de next. Tu avais redémarré ton serveur? Pour les autres logs pas beaux ça devrait être corrigé.

MaGOs92 commented 1 year ago

Yes j'avais oublié de rendre le champ "Nom de commune" requis pour poster le formulaire. ça devrait être bon!

MaGOs92 commented 1 year ago

ça marche, plutôt que de rendre le champ obligatoire je l'ai supprimé pour les communes. En faite on récupère déjà le code département lors du call à l'API géo, donc en plus ça évite les erreurs de saisie. Sophie m'a confirmé que le périmètre d'une commune c'est tjs son département