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

Notifier normalement les MP de sanction #6323

Merged
merged 3 commits into from
Jul 30, 2022
Merged

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented May 30, 2022

Fix #5746.

Actuellement, quand un utilisateur est sanctionné, un email et un MP sont envoyés mais le MP n'est pas notifié convenablement (càd qu'il n'y a pas de bulle rouge à côté de l'icône de messagerie). Cette PR effectue quelques changements qui permettent de mieux gérer ce cas et de bien notifier l'utilisateur lorsqu'il est sanctionné.

Contrôle qualité

  • Se connecter en tant que staff
  • Appliquer une sanction à user1 (par exemple, LS ou LS temporaire)
  • Se déconnecter
  • Se connecter en tant que user1
    Résultat attendu: Une notification (bulle rouge) est présente à côté de l'icône de messagerie, notifiant l'utilisateur de sa sanction.
  • Se déconnecter
  • Se connecter en tant que staff
  • Retirer la sanction à user1
  • Se déconnecter
  • Se connecter en tant que user1
    Résultat attendu: Une notification (bulle rouge) est présente à côté de l'icône de messagerie, notifiant l'utilisateur de la levée de sa sanction.

@coveralls
Copy link

coveralls commented May 30, 2022

Coverage Status

Coverage increased (+0.007%) to 88.2% when pulling 2555e03 on Migwel:issue5746 into c434629 on zestedesavoir:dev.

@Arnaud-D
Copy link
Contributor

En dehors de la remarque ci-dessus à régler, ça marche. ^^

Et félicitations, les notifications ce n'est pas le sujet le plus facile du code et ce que tu as fait me semble bien plus propre que ce qu'on avait avant !

@Arnaud-D Arnaud-D enabled auto-merge (squash) July 30, 2022 19:35
@Arnaud-D Arnaud-D merged commit 777e109 into zestedesavoir:dev Jul 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Notifier normalement les MP de sanction
3 participants