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.

Sign up for GitHub

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

Jump to bottom

[FO - Signalement] Détecter les doublons #2416

Merged
merged 8 commits into from
Apr 22, 2024
Merged

[FO - Signalement] Détecter les doublons #2416

merged 8 commits into from
Apr 22, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Apr 4, 2024 •
edited
Loading

Ticket

#2327

Description

Détecter un signalement déjà créé sur adresse + mail déclarant
Si tiers déclarant, proposer une liste de signalements (le cas échéant)

Changements apportés

  • Mise à jour de l'app Vue et du json questions_profile_tous.json
  • Mise à jour du composant Modal pour l'adapter aux deux types de doublons (et renommage des composants, variables, fonctions etc.)

Pré-requis

Tests

  • Faire un signalement avec même adresse et même mail déclarant qu'un signalement existant (non archivé ni cloturé)
  • Vérifier que la modale s'affiche
  • Si je clique sur recevoir un lien de suivi, vérifier qu'on est redirigé sur une nouvelle page de fin, et qu'on reçoit le mail, et qu'il n'y a pas un signalement-draft créé
  • Si je clique sur l'autre bouton, vérifier qu'on continue le parcours et que le signalement-draft est enregistré
  • Refaire le test en tiers déclarant en ayant soin d'avoir plusieurs signalements à la même adresse (adresse BAN) avec le même mail déclarant. Vérifier qu'on a bien une lsite de signalements dans la modale, et qu'on est obligé de choisir le signalement dont on souhaite recevoir le lien de suivi.
  • Vérifier que la reprise de brouillon fonctionne toujours
  • Commencer un signalement, en tant qu'OCCUPANT sur un signalement (non archivé ni cloturé) fait par un tiers, en mettant bien le même mail occupant. Récupérer le lien de suivi, et vérifier qu'il est bien envoyé sur l'adresse mail de l'occupant.

@hmeneuvrier hmeneuvrier changed the title [WIP] identify doublon #2327 [WIP] [FO - Signalement] Détecter les doublons Apr 4, 2024
@hmeneuvrier hmeneuvrier force-pushed the feature/2327-fo-signalement-detecter-les-doublons branch 2 times, most recently from cefef17 to 69ba2e8 Compare April 8, 2024 07:27
@hmeneuvrier hmeneuvrier requested review from numew and sfinx13 April 8, 2024 13:45
@hmeneuvrier hmeneuvrier changed the title [WIP] [FO - Signalement] Détecter les doublons [FO - Signalement] Détecter les doublons Apr 8, 2024
@hmeneuvrier hmeneuvrier marked this pull request as ready for review April 8, 2024 13:46
numew
numew reviewed Apr 9, 2024
View reviewed changes
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.

Quelques commentaires de code.

1 tests KO :

  • La reprise de brouillon ne fonctionne plus (on passe null en paramètre de la route)
    Screenshot 2024-04-09 at 11-25-28 Symfony Profiler

@hmeneuvrier hmeneuvrier force-pushed the feature/2327-fo-signalement-detecter-les-doublons branch from 28b216e to 9f95ec2 Compare April 10, 2024 08:44
@hmeneuvrier hmeneuvrier force-pushed the feature/2327-fo-signalement-detecter-les-doublons branch from b592919 to 86b812d Compare April 16, 2024 08:44
numew
numew approved these changes Apr 16, 2024
View reviewed changes
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 et relecture ok (une petite remarque de code facultative)

@hmeneuvrier hmeneuvrier force-pushed the feature/2327-fo-signalement-detecter-les-doublons branch from b11d9d6 to a572810 Compare April 16, 2024 13:00
@hmeneuvrier hmeneuvrier marked this pull request as draft April 16, 2024 13:30
@hmeneuvrier hmeneuvrier force-pushed the feature/2327-fo-signalement-detecter-les-doublons branch from 8497cde to 6434ca6 Compare April 19, 2024 10:35
@hmeneuvrier hmeneuvrier marked this pull request as ready for review April 19, 2024 13:16
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
2 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 reviewed Apr 22, 2024
View reviewed changes
Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

Des petits détails, pas utile de bloquer la PR
Lecture OK + Test OK

src/Controller/FrontSignalementController.php
'already_exists' => true,
'type' => 'signalement',
'signalements' => $signalements,
'uuid_draft' => $existingSignalementDraft?->getUuid() ?? null,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pas utile ici d'envoyer l'uuid_draft.

src/Controller/FrontSignalementController.php
SignalementDraftRepository $signalementDraftRepository,
SignalementRepository $signalementRepository,
): Response {
Copy link
Collaborator

Choose a reason for hiding this comment

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

JsonResponse

src/Controller/FrontSignalementController.php
NotificationMailerRegistry $notificationMailerRegistry,
Signalement $signalement,
Request $request
): Response {
Copy link
Collaborator

Choose a reason for hiding this comment

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

JsonResponse

@sfinx13 sfinx13 merged commit 6327015 into develop Apr 22, 2024
2 of 3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2327-fo-signalement-detecter-les-doublons branch April 26, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfinx13 sfinx13 sfinx13 requested changes

@numew numew numew approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hmeneuvrier @sfinx13 @numew