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

[BO - Signalement] Un admin partenaire ne peut plus ajouter des suivis sur un signalement sans avoir accepté l'affectation #2732

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#2729

Description

Un admin partenaire pouvait ajouter des suivi sur un signalement qu'il n'avait pas accepté. Correction de ce bug

Changements apportés

  • Durcissement du SuiviVoter
  • Utilisation du voter COMMENT_CREATE dans le twig

Pré-requis

Tests

  • Tester en SA : on ne peut créer un suivi que sur un signalement au statut en cours
  • Tester en RT : idem, même si on est auto-affecté sur le signalement
  • Tester en admin partenaire : on ne peut créer un suivi que sur un signalement au statut en cours et dont on a accepté l'affectation
  • Tester en agent : idem admin partenaire

@hmeneuvrier hmeneuvrier changed the title [BO - Signalement] Un admin partenaire peut ajouter des suivis sur un signalement sans avoir accepté l'affectation [BO - Signalement] Un admin partenaire ne peut plus ajouter des suivis sur un signalement sans avoir accepté l'affectation Jun 25, 2024
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.

Ca me semble ok, par contre on peux supprimer la fonction canView du voter et la constante associé qui ne sont pas utilisé

@hmeneuvrier
Copy link
Collaborator Author

par contre on peux supprimer la fonction canView du voter et la constante associé qui ne sont pas utilisé

@numew fait

Copy link

sonarcloud bot commented Jun 25, 2024

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.

Ok

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 test + OK

@sfinx13 sfinx13 merged commit 9336a90 into main Jun 27, 2024
3 checks passed
@sfinx13 sfinx13 deleted the hotfix/2729-bo---signalement-no-suivi-until-acceptation branch June 27, 2024 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants