Mehrere Benachrichtigungen über dieselbe Antwort erhalten

Fehler
1

Das erste Mal, als ich eine zweite Benachrichtigung für dieselbe Antwort bemerkte, ohne dass im Bearbeitungsvorgang ein Link, ein Zitat oder eine Erwähnung hinzugefügt wurde, war nach der Bearbeitung unter Topics from some categories do not appear on /latest - #36 by JammyDodger. Dieser Fall unterscheidet sich geringfügig von meinen unten aufgeführten Reproduktionsschritten, aber ich denke, das zugrunde liegende Problem ist dasselbe.

Der zweite Beitrag, bei dem dies geschah, war unter Messages section for sidebar - #13 by nathank. Es war ähnlich: Die Bearbeitung fügte nichts hinzu, was eine Benachrichtigung ausgelöst hätte – die Zitate waren beide schon vorher vorhanden – und trotzdem wurde ich ein zweites Mal benachrichtigt.

Hier sind die Schritte zur Reproduktion, die ich gefunden habe und die funktioniert haben [1]

Sie benötigen 3 Benutzer: OP, notifiedUsers, spammer

  1. OP erstellt ein Thema
  2. benachrichtigter Benutzer antwortet
  3. OP antwortet auf den Beitrag des benachrichtigten Benutzers
    benachrichtigter Benutzer wird über die Antwort benachrichtigt (erwartet)
  4. Spammer antwortet auf den Beitrag des benachrichtigten Benutzers. Die Antwort enthält einen Link zu einem anderen Beitrag des benachrichtigten Benutzers und ein Zitat des Beitrags, auf den geantwortet wird. (optional: Sie können den benachrichtigten Benutzer auch @erwähnen)
    benachrichtigter Benutzer wird über die Antwort benachrichtigt (erwartet)
    [Falls Sie eine @Erwähnung hinzugefügt haben, bezieht sich die Benachrichtigung auf die @Erwähnung (erwartet)]
  5. benachrichtigter Benutzer liest die neuen Antworten, um die Benachrichtigungen als gelesen zu markieren, und navigiert woanders hin, damit wir eine Benachrichtigung nicht verpassen.
  6. Spammer bearbeitet die Antwort und korrigiert einen Tippfehler (oder fügt edit1 hinzu)
    benachrichtigter Benutzer wird über das Zitieren benachrichtigt (unerwartet, sie wurden bereits über diese Antwort benachrichtigt und das Zitat war schon vorher da, es besteht kein Grund, sie erneut zu informieren)
  7. Spammer bearbeitet die Antwort erneut, um einen weiteren Tippfehler zu korrigieren (oder fügt edit2 hinzu)
    benachrichtigter Benutzer wird über die Verlinkung benachrichtigt (unerwartet, sie wurden bereits über diese Antwort benachrichtigt und der Link war schon vorher da, es besteht kein Grund, sie erneut zu informieren)

Das Video zeigt nur die letzten Schritte 5-7. Spammer ist links, benachrichtigter Benutzer ist rechts

  1. zumindest meistens, manchmal löst selbst das Hinzufügen einer @Erwähnung in der Bearbeitung keine neue Benachrichtigung aus ↩︎

2 „Gefällt mir“
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

Dies wird funktionieren, aber ich mache mir etwas Sorgen, da hier noch andere Benachrichtigungen durchrutschen können. (z.B. Plugin-Benachrichtigungen, die wir möglicherweise unterdrücken möchten)

@lindsey Hier gibt es eine Produktfrage: Wann sollten wir eine Benachrichtigung unterdrücken?

Ist die kleine Korrektur ein Schritt nach vorn?