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] Signalement refusé par RT #1777

Merged
merged 4 commits into from
Sep 29, 2023

Conversation

hmeneuvrier
Copy link
Collaborator

@hmeneuvrier hmeneuvrier commented Sep 26, 2023

Ticket

#1684

Description

Prise en compte du statut "Refusé" à divers endroits :

  • affichage de ce statut dans la liste des signalements (il n'y avait rien)
  • ajout du statut "Refusé" dans le filtre par statut
  • exclusion des signalements au statut "Refusés" dans le compte des signalements sans suivi depuis 30 jours

Changements apportés

  • Modification du SuiviRepository pour la prise en compte du statut dans les signalements sans suivi depuis 30 jours
  • Modification du twig table_result pour afficher le statut
  • Modification du twig de base pour ajouter le statut Refusé dans les filtres

Pré-requis

Tests

  • En superAdmin, aller sur le tableau de bord et vérifier le nombre de signalements sans suivi depuis 30 jours
  • Retenir ce nombre et cliquer sur cette carte pour afficher la liste
  • Modifier le statut d'un de ces signalements en bdd et lui affecter le statut 8
  • Mettre à jour la liste -> le signalement ne doit plus apparaitre
  • Sur le tableau de bord, le widget doit avoir 1 de moins sur la carte
  • Afficher maintenant les nouveaux signalements.
  • En ouvrir 1 et le refuser
  • Revenir sur la liste, et filtrer les signalements par statut "refusé"
  • Il doit y avoir les deux signalements précédents (ou plus s'il y en avait déj)
  • la colonne statut ne doit pas être vide, il doit être écrit "Refusé"

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.

@hmeneuvrier tu peux ajouter un signalement refusé dans les fixtures. Merci :-)

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.

J'ai du supprimer le cache du dashboard pour voir l'affichage, mais ça fonctionne.
OK avec la demande de Saidi ! :)

@hmeneuvrier
Copy link
Collaborator Author

@sfinx13 @emilschn j'ai rajouté des fixtures

@sonarcloud
Copy link

sonarcloud bot commented Sep 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sfinx13 sfinx13 merged commit d278523 into develop Sep 29, 2023
4 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/1684-bo-signalement-status-refused branch October 2, 2023 14:10
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.

3 participants