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

[FO - Signalement] Reprise de brouillon de signalement #2266

Merged
merged 9 commits into from
Mar 12, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Feb 20, 2024

Ticket

#87

Description

Flow pour la reprise de brouillon

  • Je recommence une saisie
  • Je saisis mes coordonnées
  • Je clique sur Suivant
  • On affiche une modale :
    Titre : Reprendre la saisie
    Texte :
    Il semblerait que vous avez commencé à remplir un signalement pour le logement situé {{adresse}}, le {{date}} à {{heure dernière modif}}. Vous pouvez récupérer les infos de ce signalement et reprendre ou vous vous êtes arrêté. Souhaitez-vous reprendre le signalement ?
    Boutons :
    • Oui, reprendre le signalement
    • Non, faire un nouveau signalement

Conséquences

  • Si oui
    • On ferme la modale et on affiche une page de validation (même modèle que celle qui dit "Merci pour votre signalement") qui dit Un email a été envoyé à votre adresse. Cliquez sur le lien pour reprendre votre signalement !
    • Le user clique sur le lien dans l'email et accède au signalement avec le brouillon chargé
  • Si non
    • On crée un nouveau brouillon pour le signalement en cours
    • Le premier brouillon est archivé
    • On ferme la modale et on passe à l'écran suivant

Changements apportés

  • Création d'une migration pour ajouter un identification_hash à tous les signalement_draft (et ajout des fonctions nécessaires dans l'entité)
  • Ajout d'un statut ARCHIVE pour les signalementDraft, et utilisation de l'enumTrait
  • Création d'un nouveau mail ContinueFromDraft
  • Dans le json du formulaire, ajout d'une étape draft_mail accessible hors navigation courante (si on choisit de reprendre un brouillon)
  • Création de 3 nouvelles routes dans le controller :
  1. une route check_signalement_draft_existe pour vérifier si un signalementDraft avec le même hash existe déjà
  2. une route send_mail_continue_from_draft pour envoyer le mail avec le lien vers le brouillon précédent
  3. une route archive_draft pour archiver un brouillon existant
  • Dans le formulaire, création d'un composant SignalementFormModalContinueFromDraft

Pré-requis

make load-migrations

Tests

  • Commencer un signalement
  • Arrêter à un moment donné après les coordonnées
  • Recommencer un signalement avec exactement la même adresse et le même e-mail
  • Vérifier que la modale s'ouvre, choisir de reprendre le brouillon précédent
  • Vérifier qu'on arrive bien sur une page "un email a été envoyé à votre adresse"
  • Vérifier en base qu'un deuxième brouillon n'a pas été créé
  • Vérifier qu'on a bien reçu un mail , cliquer sur le lien
  • Vérifier qu'on reprend bien le brouillon on était la première fois, avancer un peu le signalement
  • Arrêter le signalement
  • Recommencer du début avec la même adresse mail et la même adresse
  • On a de nouveau la modale, choisir maintenant de ne pas reprendre le brouillon
  • Vérifier qu'on passe bien à la page suivante
  • Vérifier en base que le premier brouillon a été archivé et qu'un deuxième brouillon a été créé
  • Commencer un autre signalement qui n'est pas à la même adresse, ou pas avec la même adresse mail
  • Vérifier que tout se déroule normalement sans affichage de la modale

@hmeneuvrier hmeneuvrier force-pushed the feature/87-identification-signalement-doublon branch 2 times, most recently from 1200ea7 to 2a8fb0b Compare February 26, 2024 16:31
@hmeneuvrier hmeneuvrier changed the title [WIP] [FO - Signalement] Reprise de brouillon de signalement Feb 28, 2024
@hmeneuvrier hmeneuvrier marked this pull request as ready for review February 28, 2024 13:59
Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

2 retours de code
Tests ok !
C'est cool :)

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.

Tests ok, mais quelques trucs à revoir et une interrogation sur l'utilité du identificationHash

migrations/Version20240212150918.php Show resolved Hide resolved
src/Controller/FrontNewSignalementController.php Outdated Show resolved Hide resolved
assets/vue/components/signalement-form/requests.ts Outdated Show resolved Hide resolved
assets/vue/components/signalement-form/requests.ts Outdated Show resolved Hide resolved
src/Entity/SignalementDraft.php Outdated Show resolved Hide resolved
src/Controller/FrontNewSignalementController.php Outdated Show resolved Hide resolved
src/Controller/FrontNewSignalementController.php Outdated Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the feature/87-identification-signalement-doublon branch from 40abe09 to 1264f22 Compare March 4, 2024 10:43
src/Entity/SignalementDraft.php Outdated Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the feature/87-identification-signalement-doublon branch from 69d244e to 05dcfc2 Compare March 6, 2024 10:23
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.

Test et relecture OK

@sfinx13 sfinx13 marked this pull request as draft March 7, 2024 08:55
@sfinx13
Copy link
Collaborator

sfinx13 commented Mar 7, 2024

Vu en stand-up en attente de la prochaine MEP

@sfinx13 sfinx13 marked this pull request as ready for review March 11, 2024 08:10
Copy link
Collaborator

@emilschn emilschn 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 (avec commit de modification d'affichage)

Copy link

sonarcloud bot commented Mar 11, 2024

Quality Gate Passed Quality Gate passed

Issues
9 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@sfinx13 sfinx13 merged commit 07fdf09 into develop Mar 12, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/87-identification-signalement-doublon branch March 15, 2024 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants