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

Empêche les notifications lorsqu'un nouveau topic est créé dans un forum auquel l'utilisateur n'a pas accès #6363

Merged
merged 2 commits into from
Aug 12, 2022

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented Jul 28, 2022

Fix #6218

Contrôle qualité

Scénario 1:

  • Se connecter avec user
  • S'abonner à un forum
  • Se déconnecter
  • Se connecter à l'interface admin et rendre ce forum uniquement accessible au staff (cela peut se faire directement via la db également via un insert dans forum_forum_groups )
  • Se connecter avec staff
  • Créer un topic dans la zone staff dans ce forum
  • Se déconnecter
  • Se connecter avec user
    Résultat attendu: Aucune notification n'a été générée car le topic a été créé dans un forum auquel user n'a pas accès

Test de non-régression:

  • Se connecter avec user
  • S'abonner à un forum quelconque
  • Se déconnecter
  • Se connecter avec staff
  • Créer un topic dans ce forum
  • Se déconnecter
  • Se connecter avec user
    Résultat attendu: Une notification a bien été créée concernant ce topic, car il a été créé dans un forum auquel user a accès

@coveralls
Copy link

coveralls commented Jul 28, 2022

Coverage Status

Coverage decreased (-10.7%) to 77.51% when pulling 30a5673 on Migwel:issue6218 into b6576ca on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) August 12, 2022 15:28
@Arnaud-D Arnaud-D merged commit 85ad533 into zestedesavoir:dev Aug 12, 2022
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.

Notification persistante dans un cas complexe
3 participants