search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Masquer un commentaire ne supprime pas les notifications associées #6399

Open Situphen opened 2 years ago

Situphen commented 2 years ago

Description du bug

Lorsqu'un membre poste un commentaire sur un contenu, tous les membres abonnés à ce contenu reçoivent une notification. Si ce membre ou un modérateur décide de masquer ce commentaire, la notification n'est plus pertinente et devraient alors être marquée comme lue. Ce n'est pas le cas actuellement pour les commentaires !

Ce fonctionnement est déjà en place pour les messages du forum, il doit donc déjà y avoir des bouts de code qui s'occupent de ça quelque part.

Comment reproduire ?

  1. Bob est abonné à un contenu
  2. Alice écrit un commentaire sur ce contenu
  3. Alice décide finalement de le masquer
  4. Bob se retrouve alors avec une notification qui pointe vers le message masqué

Comportement attendu

Bob n'a aucune notification.

Informations techniques

Arnaud-D commented 1 year ago

On n'a qu'un seul abonnement pour le contenu et ses commentaires, non ? A-t-on un risque de supprimer une notification liée à la mise à jour du contenu ?

Situphen commented 1 year ago

On a bien un seul abonnement pour ces deux cas-là mais ça ne devrait pas poser de soucis je pense car on peut filtrer les abonnements qui concernent un commentaire en particulier. Je suppose que dans le cas d'une mise à jour ce champ (note) doit être vide.

https://github.com/zestedesavoir/zds-site/blob/e78c57a139f29c507fca5dedd2418fb3cdfbee3b/zds/tutorialv2/models/database.py#L1255-L1269