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

Supprime notifications forum inaccessible (#6186) #6196

Conversation

ChantyTaguan
Copy link
Contributor

Ajout d'une migration manuelle désactivant les anciens abonnements aux forum nécessitant d'être dans un groupe pour y avoir accès.

Lorsque l'on quitte un groupe, les abonnements aux sujets et forums accessibles à ce groupe sont désactivés. Cependant, lorsque cela a été mis en place, certains abonnements étaient déjà caduques et sont donc restés en place. Une mirgation a été créée pour nettoyer les abonnements à un sujet, mais les abonnements à un forum ont été oubliés. Cette pull request répare cet oubli.

Quelques petites améliorations supplémentaires ont été faites dans la foulée :

Numéro du ticket concerné : #6186

La PR nécessite une migration de la DB.

Contrôle qualité

Avant de migrer la DB

  • recréer une db avec make new-db
  • vérifier qu'un forum "staff only" apparait pour les staff et les admins, mais pas pour les simples users
  • s'abonner au forum "Staff only" avec un utilisateur, ainsi qu'à un sujet de ce forum (en en créant un par exemple)
  • provoquer des notification pour cet utilisateur en allant créer un sujet, et en répondant au sujet auquel il est abonné avec un autre utilisateur
  • retirer l'utilisateur du groupe "staff" et vérifier que les notifications en question ont bien disparu. Vérifier qu'il n'est plus abonné au forum "staff only", ni au sujet de ce forum.

Migration de la DB

Il faut d'abord aller réactiver l'abonnement au forum "staff only" créé précédemment (via l'administration Django)

Ensuite :

  • Créer un sujet dans le forum "staff only" et constater que l'utilisateur reçoit une notification pour un sujet auquel il n'a pas accès
  • Lancez make migrate-db
  • Vérifier que la notification a disparu que la création d'un nouveau sujet n'en recrée pas une nouvelle

@coveralls
Copy link

coveralls commented Oct 19, 2021

Coverage Status

Coverage decreased (-0.001%) to 87.019% when pulling 3e0d688 on ChantyTaguan:bugfix-6186-notifications-forum-inaccessible into 44a96f4 on zestedesavoir:dev.

@Arnaud-D Arnaud-D linked an issue Oct 20, 2021 that may be closed by this pull request
@Arnaud-D Arnaud-D added this to En développement in Suivi des PR via automation Oct 20, 2021
@Arnaud-D Arnaud-D moved this from En développement to En attente de QA in Suivi des PR Oct 20, 2021
@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 20, 2021
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

Rapport de QA

Tout est OK ✔️

J'ai juste du "désactiver" la migration que tu as introduite pour faire make new-db, sinon le make migrate-db n'aurait ensuite plus de migration à appliquer.

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Dec 28, 2021
@philippemilink philippemilink enabled auto-merge (squash) December 28, 2021 13:36
@philippemilink philippemilink merged commit 915c7d1 into zestedesavoir:dev Dec 28, 2021
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Dec 28, 2021
@Situphen Situphen deleted the bugfix-6186-notifications-forum-inaccessible branch December 28, 2021 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

Notifications sur des sujets auxquels on n'a plus accès
4 participants