Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO - Signalement] Réinitialisation de la géolocalisation quand édition de l'adresse #2588

Merged
merged 4 commits into from
May 30, 2024

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented May 24, 2024

Ticket

#2342

Description

Suite aux retours d'Arnaud, quelques modifications nécessaires pour les adresses hors BAN.

Changements apportés

Dans le BO,

  • quand on édite une adresse à la main, il faut l'alerte pour les autres agents
  • mise à jour du code Insee, de la géolocalisation et du nom de la ville, en cas de modification du couple code postal / ville

Dans le FO,

  • modification mineure sur l'indication de la saisie d'adresse
  • mise à jour de la géolocalisation et du nom de la ville, en cas de modification du couple code postal / ville

Tests

  • Vérifier la description du champ sur le FO
  • Vérifier les différents comportements en éditant une adresse via le BO (choisir dans la liste, modifier un ou plusieurs champs, etc.)

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour la modif sur le front, OK.
Par contre sur le BO il y a un truc que je ne comprends pas. J'ai modifié une adresse en mettant une adresse qui n'existe pas encore sur la BAN.
J'ai bien le code insee, et j'ai bien le suivi.
Mais comme geoloc, j'ai
{"lat": null, "lng": null}
Il me semblait qu'on devait prendre la geoloc du centre de la ville, on n'avait pas dit ça ?

@emilschn
Copy link
Collaborator Author

emilschn commented May 29, 2024

Pour la modif sur le front, OK. Par contre sur le BO il y a un truc que je ne comprends pas. J'ai modifié une adresse en mettant une adresse qui n'existe pas encore sur la BAN. J'ai bien le code insee, et j'ai bien le suivi. Mais comme geoloc, j'ai {"lat": null, "lng": null} Il me semblait qu'on devait prendre la geoloc du centre de la ville, on n'avait pas dit ça ?

On n'a pas fait ça en front. Je viens de vérifier, je n'étais plus sûr. (voir SignalementFormAddress, ligne 197)
Mais du coup, je ne me souviens pas d'une prise de décision à ce niveau..?
Le code Insee était indispensable, mais ça me fait bizarre de mettre le point sur le centre de la ville qui n'a sûrement rien à voir.

@emilschn
Copy link
Collaborator Author

@hmeneuvrier c'est corrigé !

Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok pour moi relecture et tests

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok test et relecture

@numew numew merged commit 7be25fc into develop May 30, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the fix/2342-out-ban branch June 11, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants