Ricezione di notifiche multiple per la stessa risposta

Bug
2

In 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

Questo funzionerà, ma sono un po’ preoccupato perché ci sono altre notifiche che potrebbero passare inosservate qui. (ad esempio le notifiche dei plugin che potremmo voler sopprimere)

@lindsey qui c’è una domanda di prodotto: quando dovremmo sopprimere una notifica?

Suppongo che la piccola correzione sia un passo avanti?