[FO - Signalement] Saisie des adresses hors-BAN

[emilschn]

Ticket

#2342

Description

Sur certains département, les adresses recherchées ne se trouvent pas dans la BAN.
On ouvre donc la possibilité aux gens de saisir directement leur adresse.
Inconvénient : le code INSEE et la géolocalisation seront faux pour ces adresses.

Pré-requis

make load-migrations

npm run watch

Tests

• Faire un signalement : vérifier qu'on a accès aux différents champs pour l'adresse occupant et l'adresse propriétaire
• BO : Vérifier qu'on retrouve bien les adresses modifiées
• BO : Vérifier qu'on peut éditer les champs séparément
• Si modification du code postal ou de la ville, vérifier qu'une mise à jour du code INSEE est faite
• BO : Si modification d'un des champs, vérifier qu'une alerte s'affiche
  • BO : Modifier l'adresse en tant qu'agent, et vérifier que l'alerte disparait

[hmeneuvrier]

Fonctionnement à revoir, cf discussion : https://mattermost.incubateur.net/betagouv/pl/bg6gasys5tn1djqmq5cukb6xny

[emilschn]

Retours de @numew (copier-coller de Mattermost)

• Je ne peux pas saisir une adresse librement sans avant au préalable sélectionner une adresse BAN
• Si je modifie le code postal/commune manuellement je n'ai pas la popup territoire innactif comme lors de la sélection BAN mais une erreur du validateur plus loin
• Si je retourne sur la page adresse par défaut on voit uniquement l'adresse BAN qui n'est pas la bonne si modifié manuellement
• Si Je vide le champ "Numéro et voie" j'ai un crash (en lien avec type string attendu pour null reçu sur ce champ)
• Si je rentre dans le champs que j'efface le code postal et que ejremet le même, l'adresse est enregistré comme saisie manuellement.
• Idem si je saisie manuellement une adresse exactement comme retourné par la BAN, ne pourrait on pas faire un check et si on trouve une correspindante BAN exacte ne pas garder le flag manuel ?
• De cette manière dans le BO on garderait aussi le flag aprés modification admin si pas de correspondance exacte BAN, on le supprime dans le cas inverse. (aujourd'hui on le perd même en faisant une modif manuelle via le BO)

[sfinx13]

Le wording peut être perturbant, on peut le mettre à jour et faire comme punaises
Saisir une adresse manuellement à place de Afficher tous les champs

[sfinx13]

Je ne peux pas saisir une adresse librement sans avant au préalable sélectionner une adresse BAN

On devrait s'inspirer (sans la map :-) ) de ce que fait mapbox ou a on 1 seul champ adresse (manuel ou automatique) au lieu du champ suggestion qui est obligatoire aujourd'hui + (adresse)
Je vous laisse voir https://www.mapbox.com/address-autofill

[emilschn]

@numew de mon point de vue :

Retours à corriger :

• Je ne peux pas saisir une adresse librement sans avant au préalable sélectionner une adresse BAN
• Si je modifie le code postal/commune manuellement je n'ai pas la popup territoire innactif comme lors de la sélection BAN mais une erreur du validateur plus loin
• Si Je vide le champ "Numéro et voie" j'ai un crash (en lien avec type string attendu pour null reçu sur ce champ)
• Si je retourne sur la page adresse par défaut on voit uniquement l'adresse BAN qui n'est pas la bonne si modifié manuellement

---

Je trouve les retours suivants secondaires.
Les deux premiers parce qu'ils me semblent compliqués à tester, et nécessiteraient un comportement hyper spécifique du composant. Je ne suis pas contre dans l'idée, mais ça peut être refait quand on refera la totalité du composant ?
Le troisième parce que le flag dit Cette adresse a été éditée manuellement lors de la déclaration. ; donc si on l'édite dans le BO, peu importe le résultat, mais l'état de la déclaration n'est plus importante.
A court terme, ces retours ne me semblent pas hyper gênants. On fait une sorte de patch en attendant une solution plus globalisée qu'il faudra re-réfléchir ensemble.

Si je rentre dans le champs que j'efface le code postal et que ejremet le même, l'adresse est enregistré comme saisie manuellement.
Idem si je saisie manuellement une adresse exactement comme retourné par la BAN, ne pourrait on pas faire un check et si on trouve une correspindante BAN exacte ne pas garder le flag manuel ?
De cette manière dans le BO on garderait aussi le flag aprés modification admin si pas de correspondance exacte BAN, on le supprime dans le cas inverse. (aujourd'hui on le perd même en faisant une modif manuelle via le BO)

[emilschn]

Les retours de @numew sont corrigés en attendant d'intégrer une meilleure UI plus tard.
Vous pouvez re-faire un tour :)

[emilschn]

Le wording peut être perturbant, on peut le mettre à jour et faire comme punaises Saisir une adresse manuellement à place de Afficher tous les champs

J'ai changé le picto aussi, du coup.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.5% Duplication on New Code

See analysis details on SonarCloud

[numew]

Encore un point KO pour moi :

• Quand je rentre une adresse sur un territoire non ouvert, j'ai un popup non bloquant "code postale/code insee non cohérent" et à la soumission des coordonnée une alerte bloquante du validateur
  
  

• Je relève que quoi que je fasse dans l'admin le message "Cette adresse a été éditée manuellement lors de la déclaration." reste, mais ce comportement n'est pas gênant a mon sens (je crois que je préfère)

Je propose une feature qui ferait un check sur la BAN à partir d'une adresse manuelle pour voir si on la retrouve. Objectif : avoir une idée du nombre d'adresse hors BAN présente dans nos signalement, mais je pense que c'est pas du tout prioritaire donc je pose juste l'idée ici, je créerai éventuellement le ticket correspondant.

[emilschn]

Encore un point KO pour moi :

* Quand je rentre une adresse sur un territoire non ouvert, j'ai un popup non bloquant "code postale/code insee non cohérent" et à la soumission des coordonnée une alerte bloquante du validateur
  ![01-narbonne-1](https://private-user-images.githubusercontent.com/7130780/331597446-edf760ea-5c9f-494f-a1d8-fd02ea000783.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTU5NTE4NDUsIm5iZiI6MTcxNTk1MTU0NSwicGF0aCI6Ii83MTMwNzgwLzMzMTU5NzQ0Ni1lZGY3NjBlYS01YzlmLTQ5NGYtYTFkOC1mZDAyZWEwMDA3ODMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDUxNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA1MTdUMTMxMjI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDg2ZjEzMTdjZjAyOGRlYWMzMDliMTkxZGZhYWQyMGZlOGYyNmI2M2YwZWUzMTVkZWJjYTU5M2I3YjFiODAyMyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.U_MmwBSyBCk5sqekATKxuxyPpOVXl-Nba5j8IKdR4Ao)
  ![01-narbonne-2](https://private-user-images.githubusercontent.com/7130780/331597452-f5427fe4-e341-4ee2-865b-c275248c53bd.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTU5NTE4NDUsIm5iZiI6MTcxNTk1MTU0NSwicGF0aCI6Ii83MTMwNzgwLzMzMTU5NzQ1Mi1mNTQyN2ZlNC1lMzQxLTRlZTItODY1Yi1jMjc1MjQ4YzUzYmQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDUxNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA1MTdUMTMxMjI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9Mzc4ZTk1YmZlYTUxZmE3Nzc0YmQ4YmE4ODUyZjcxNWU0YzcwMmFiODdkM2JkN2UwZjNiM2I5NzQ4MDZkYjg0OCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.-A3na-FDhxZkatrQS_pMIXOXTgwSW2PKzk_zPuFKsk0)

* Je relève que quoi que je fasse dans l'admin le message "Cette adresse a été éditée manuellement lors de la déclaration." reste, mais ce comportement n'est pas gênant a mon sens (je crois que je préfère)

Je propose une feature qui ferait un check sur la BAN à partir d'une adresse manuelle pour voir si on la retrouve. Objectif : avoir une idée du nombre d'adresse hors BAN présente dans nos signalement, mais je pense que c'est pas du tout prioritaire donc je pose juste l'idée ici, je créerai éventuellement le ticket correspondant.

Je pense qu'il te reste cette erreur parce que tu fais une recherche sur un territoire dont les communes n'ont pas été importées. Peux-tu soit importer les communes (commande update-communes), soit désactiver un territoire avec des communes importées pour voir ?

[numew]

Encore un point KO pour moi :

* Quand je rentre une adresse sur un territoire non ouvert, j'ai un popup non bloquant "code postale/code insee non cohérent" et à la soumission des coordonnée une alerte bloquante du validateur
  ![01-narbonne-1](https://private-user-images.githubusercontent.com/7130780/331597446-edf760ea-5c9f-494f-a1d8-fd02ea000783.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTU5NTE4NDUsIm5iZiI6MTcxNTk1MTU0NSwicGF0aCI6Ii83MTMwNzgwLzMzMTU5NzQ0Ni1lZGY3NjBlYS01YzlmLTQ5NGYtYTFkOC1mZDAyZWEwMDA3ODMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDUxNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA1MTdUMTMxMjI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDg2ZjEzMTdjZjAyOGRlYWMzMDliMTkxZGZhYWQyMGZlOGYyNmI2M2YwZWUzMTVkZWJjYTU5M2I3YjFiODAyMyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.U_MmwBSyBCk5sqekATKxuxyPpOVXl-Nba5j8IKdR4Ao)
  ![01-narbonne-2](https://private-user-images.githubusercontent.com/7130780/331597452-f5427fe4-e341-4ee2-865b-c275248c53bd.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTU5NTE4NDUsIm5iZiI6MTcxNTk1MTU0NSwicGF0aCI6Ii83MTMwNzgwLzMzMTU5NzQ1Mi1mNTQyN2ZlNC1lMzQxLTRlZTItODY1Yi1jMjc1MjQ4YzUzYmQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDUxNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA1MTdUMTMxMjI1WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9Mzc4ZTk1YmZlYTUxZmE3Nzc0YmQ4YmE4ODUyZjcxNWU0YzcwMmFiODdkM2JkN2UwZjNiM2I5NzQ4MDZkYjg0OCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.-A3na-FDhxZkatrQS_pMIXOXTgwSW2PKzk_zPuFKsk0)

* Je relève que quoi que je fasse dans l'admin le message "Cette adresse a été éditée manuellement lors de la déclaration." reste, mais ce comportement n'est pas gênant a mon sens (je crois que je préfère)

Je propose une feature qui ferait un check sur la BAN à partir d'une adresse manuelle pour voir si on la retrouve. Objectif : avoir une idée du nombre d'adresse hors BAN présente dans nos signalement, mais je pense que c'est pas du tout prioritaire donc je pose juste l'idée ici, je créerai éventuellement le ticket correspondant.

Je pense qu'il te reste cette erreur parce que tu fais une recherche sur un territoire dont les communes n'ont pas été importées. Peux-tu soit importer les communes (commande update-communes), soit désactiver un territoire avec des communes importées pour voir ?

En effet, il y'a quand même un petit changement de comportement qui est qu'auparavant on ne pouvait pas passer à l'étape suivante aprés avoir vu ce popup, mais je valide

[numew]

Ok

[sfinx13]

Lecture et test OK