Recebendo múltiplas notificações sobre a mesma resposta

Bug
2

Em post_alerter.rb

Mude as linhas 589-599 de:
 # linked, quoted, mentioned, chat_quoted podem ser suprimidos se você já tiver uma notificação de resposta
 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

 Para:
 # linked, quoted, mentioned, chat_quoted podem ser suprimidos se você já tiver qualquer notificação sobre esta
 postagem
 if [
      Notification.types[:quoted],
      Notification.types[:linked],
      Notification.types[:mentioned],
      Notification.types[:chat_quoted],
    ].include?(type)
   return if existing_notifications.any?
 end

Isso funcionará, mas estou um pouco preocupado porque há outras notificações que podem passar despercebidas aqui. (notificações de plugins, por exemplo, que talvez queiramos suprimir)

@lindsey, há uma questão de produto aqui: quando devemos suprimir uma notificação?

Acho que a pequena correção é um passo à frente?