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

[Bug] Bloquer page front signalement si archivé #2380

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented Mar 26, 2024

Ticket

#2378

Description

  • Route front_suivi_signalement, front_suivi_procedure et front_suivi_signalement_user_response non accessible pour des signalements archivé
  • On bloque la possibilité d'envoyer des message sur les signalement clôturé/refusé (avec message informatif)
  • Refacto (regroupement de ce qui concerne le suivi de procédure sans la route correspondante)
  • Ajout et adaptation des tests sur cette partie

Tests

  • Vérifier qu'on ne peux pas accéder à la page de suivi d'un signalement archivé
  • Vérifier qu'on ne peux pas envoyer de messages sur des signalement Refusé/Cloturé
  • Vérifier qu'on peux accéder à la page suivi pour les autre statuts (et déposé commentaire)

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.

Ce cas d'usager mériterai un test

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.

Relecture OK, et tests OK (j'ai testé aussi le fonctionnement d'un arrêt ou poursuite de précédure sur signalement en cours pour vérifier que ça marchait toujours bien)

Il faut juste gérer les signalementts refusés :

Pour les refusés, on affiche la page sans possibilité d'envoyer un message + un message d'info Votre signalement a été refusé, vous ne pouvez plus envoyer de messages.

@numew
Copy link
Collaborator Author

numew commented Mar 29, 2024

@sfinx13 @hmeneuvrier retours traités

Copy link

sonarcloud bot commented Mar 29, 2024

Quality Gate Passed Quality Gate passed

Issues
7 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

@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.

Tout retesté, tout relu, c'est ok pour moi

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.

Lecture et test OK

@sfinx13 sfinx13 merged commit 52a807c into main Apr 3, 2024
3 checks passed
@sfinx13 sfinx13 deleted the hotfix/2378-lock-front-archived-signalement branch April 3, 2024 09:43
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