Receiving multiple notifications about the same reply

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

This will work but I do worry a bit cause there are other notifications that can slip through here. (plugins notifications for example that are ones we may want to suppress)

@lindsey there is a bit of a product question here, when should we suppress a notification?

I guess the small fix is a step forward?

https://github.com/discourse/discourse/pull/37132