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 - Form - nouvelle étape] Renvoyer les signalements "vides" vers une nouvelle page #2568

Merged

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented May 16, 2024

Ticket

#2363

Description

Au dépôt d'un signalement, si le signalement n'a aucun désordre, renvoyer l'usager vers une page l'invitant à compléter son signalement ou l'abandonner.

Changements apportés

  • Ajout d'une nouvelle page signalement_incomplet dans les json
  • Dans l'app, vérifier qu'on a bien la référence su signalement créé lorsqu'on navigue vers la page "confirmation_signalement" sinon on redirige vers la nouvelle page
  • Création 'une action archive pour les boutons du formulaire et remaniement de la fonction archiveDraft (pour ne pas la limiter à la détection de doublon)
  • Dans le signalementBuilder on vérifie que le signalement créé a des désordres et un score supérieur à 0 pour le builder, sinon on renvoie null et on n'enregistre pas le signalement en base (et on ne change pas le statut du draft)

Pré-requis

npm run watch

Tests

  • Faire un signalement sans sélectionner aucun désordres
  • A la validation, vérifier qu'on arrive sur une page "Votre signalement est incomplet" (et qu'aucun signalement n'a été créé en base, et que le draft est toujours "EN_COURS"
  • Si on clique sur "compléter mon signalement", vérifier qu'on revient à la partie type et composition
  • Si on clique sur "abandonner" vérifier qu'on revient sur la page d'accueil, qu'aucun signalement n'est créé en base et que le draft est archivé
  • Essayer avec différent profils
  • Faire une TNR sur la détection de doublon de draft et l'archivage

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.

Tests OK pour moi !
Un mini-retour pour ajouter un com :)

@hmeneuvrier hmeneuvrier force-pushed the feature/2363-fo-form-nouvelle-etape-signalement-incomplet branch from 06fbb9d to 28785bb Compare May 17, 2024 14:54
Copy link

sonarcloud bot commented May 17, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
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

@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

@numew numew merged commit 3f3acf3 into develop May 17, 2024
3 checks passed
@hmeneuvrier hmeneuvrier deleted the feature/2363-fo-form-nouvelle-etape-signalement-incomplet branch June 11, 2024 10:27
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