Ricevere notifiche multiple per la stessa risposta

Bug
1

La prima volta che ho notato una seconda notifica per la stessa risposta senza che venissero aggiunti link, citazioni o menzioni nella modifica è stato dopo la modifica su Topics from some categories do not appear on /latest - #36 by JammyDodger. Questo caso è leggermente diverso dai miei passaggi di riproduzione di seguito, ma penso che il problema di fondo sia lo stesso.

Il secondo post in cui è successo è stato su Messages section for sidebar - #13 by nathank. Era simile: la modifica non ha aggiunto nulla che avrebbe generato una notifica - le citazioni erano entrambe presenti prima - e tuttavia sono stato nuovamente notificato.

Ecco i passaggi per riprodurre che ho trovato che hanno funzionato [1]

Hai bisogno di 3 utenti: OP, notifiedUsers, spammer

  1. OP crea un argomento
  2. L’utente notificato risponde
  3. OP risponde al post dell’utente notificato
    l’utente notificato viene notificato della risposta (previsto)
  4. Lo spammer risponde al post dell’utente notificato. La risposta contiene un link a un altro post dell’utente notificato e una citazione del post a cui si risponde. (facoltativo: puoi anche @menzionare l’utente notificato)
    l’utente notificato viene notificato della risposta (previsto)
    [Nel caso in cui tu abbia aggiunto una @menzione, la notifica riguarda la @menzione (previsto)]
  5. L’utente notificato legge le nuove risposte per contrassegnare le notifiche come lette e naviga altrove in modo da non perdere una notifica.
  6. Lo spammer modifica la risposta e corregge un errore di battitura (o aggiunge edit1)
    l’utente notificato viene notificato di essere stato citato (inaspettato, era già stato notificato di questa risposta e la citazione era già lì, non c’è bisogno di dirglielo)
  7. Lo spammer modifica nuovamente la risposta per correggere un altro errore di battitura (o aggiunge edit2)
    l’utente notificato viene notificato di essere stato linkato (inaspettato, era già stato notificato di questa risposta e il link era già lì, non c’è bisogno di dirglielo)

Il video mostra solo gli ultimi passaggi 5-7. Lo spammer è a sinistra, l’utente notificato a destra

  1. almeno la maggior parte delle volte, a volte anche l’aggiunta di una @menzione nella modifica non attiva una nuova notifica ↩︎

2 Mi Piace
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?