zds-site icon indicating copy to clipboard operation
zds-site copied to clipboard

Masquer un commentaire ne supprime pas les notifications associées

Open Situphen opened this issue 2 years ago • 2 comments

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

  • OS: Tous
  • Navigateur: Tous
  • Version/environnement de zds: Production et version locale

Situphen avatar Oct 09 '22 12:10 Situphen

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 ?

Arnaud-D avatar Oct 15 '22 16:10 Arnaud-D

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

Situphen avatar Oct 15 '22 23:10 Situphen