Recevoir plusieurs notifications pour la même réponse

Bug
2

Dans post_alerter.rb

Change lines 589-599 from:
 # linked, quoted, mentioned, chat_quoted may be suppressed if you already have a reply notification
 if [
      Notification.types[:quoted],
      Notification.types[:linked],
      Notification.types[:mentioned],
      Notification.types[:chat_quoted],
    ].include?(type)
   if existing_notifications.find { |n| n.notification_type == Notification.types[:replied] }
     return
   end
 end

 To:
 # linked, quoted, mentioned, chat_quoted may be suppressed if you already have any notification about this 
 post
 if [
      Notification.types[:quoted],
      Notification.types[:linked],
      Notification.types[:mentioned],
      Notification.types[:chat_quoted],
    ].include?(type)
   return if existing_notifications.any?
 end

Cela fonctionnera, mais je m’inquiète un peu car il y a d’autres notifications qui pourraient passer ici. (les notifications des plugins par exemple, que nous pourrions vouloir supprimer)

@lindsey il y a une question de produit ici, quand devrions-nous supprimer une notification ?

Je suppose que le petit correctif est un pas en avant ?