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

[Signalement] Modification des événements qui mettent à jour le code Insee de l'adresse des occupants #2616

Merged
merged 3 commits into from
May 31, 2024

Conversation

emilschn
Copy link
Collaborator

Ticket

#2601

Description

Suite à la possibilité donnée aux usagers et agents de modifier à la main les adresse, l'UX n'était pas idéale pour vérifier et mettre à jour le code Insee en cas d'édition manuelle.

Changements apportés

  • Côté FO : la mise à jour de la géolocalisation et la vérification de l'ouverture du territoire se fait au clic sur le bouton Suivant
    • J'ai donc ajouté un nouveau type d'action dans le json checkloc qui vient remplacer save sur le premier écran
    • J'ai déplacé la modale de territoire fermé à la racine de l'app plutôt que dans le composant d'adresse : c'est l'app elle-même qui fait cette vérification
  • Côté BO : cela se fait à la validation du formulaire, côté back
    • le js ne sert plus qu'à signaler qu'une modification manuelle a été faite ou non

Pré-requis

Pour avoir des messages corrects : make console app="update-communes"

Tests

  • FO : utiliser le champ de recherche d'adresse, et vérifier le comportement avec un territoire ouvert ou non
  • FO : modifier une adresse déjà pré-remplie et vérifier
  • FO : démarrer et écrire directement une adresse à la main, et vérifier
  • FO : vérifier le comportement avec une ville mal ortographiée
  • BO : faire les mêmes 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.

C'est tout bon :)

@@ -193,6 +214,40 @@ export default defineComponent({
}
}
},
handleValidateAddress (requestResponse: any) {
// Si le code postal / la commune ont été édités à la main, on valide l'ouverture du territoire
Copy link
Collaborator

Choose a reason for hiding this comment

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

je ne comprends pas ce commentaire

Copy link

sonarcloud bot commented May 31, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
3.4% Duplication on New Code (required ≤ 3%)

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 tests et relecture.
juste cette bizarrerie de rafraichissement navigateur sur le BO

@hmeneuvrier hmeneuvrier merged commit 27f0399 into develop May 31, 2024
2 of 3 checks passed
@hmeneuvrier hmeneuvrier deleted the fix/2601-ban-event 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

3 participants