Versehentliche E-Mail-/Benachrichtigungsflut beim Versuch, Topics lautlos zwischen Kategorien zu verschieben

Gerade eben habe ich mir die Finger verbrannt, als ich 30 Themen zwischen Kategorien verschoben habe, was zu einem E-Mail-Sturm bei meinen wertvollsten Nutzern führte.

Mir ist aufgefallen, dass sich nach dem letzten Update der Text für die Funktion, die die Unterdrückung von Benachrichtigungen ermöglicht, wenn eine per Massenauswahl ausgewählte Gruppe von Themen ihre Kategorie ändert geändert hat. Es zeigt jetzt Folgendes an:

Ich habe das nicht angekreuzt, und es hat sie wie verrückt angemailt! Die Zielkategorie ist als Beim ersten Beitrag beobachten konfiguriert, aber vor dieser Änderung wären diese unterdrückt worden.

Was geht hier vor sich?

5 „Gefällt mir“

Entschuldigung dafür, @nathank :frowning:

Ich habe versucht, dies lokal zu reproduzieren, und die Abdeckung unserer Spezifikationen verbessert, um sicherzustellen, dass wir diesen Fall behandelt haben…

… aber alles scheint wie erwartet zu funktionieren :thinking:

Haben Sie vielleicht eine Möglichkeit, Reproduktionsschritte zu finden?

1 „Gefällt mir“

@zogstrip Ich sehe keine Testabdeckung für diesen Fall.

Meiner Lektüre nach decken die Spezifikationen unter describe "silent option" nur die Szenarien ab, in denen das verschobene Thema bereits beobachtet wird.

Der hier beschriebene Fall betrifft das Szenario, in dem die Zielkategorie als „beobachtet“ oder „beim ersten Beitrag beobachten“ markiert ist.

@nathank, meines Wissens haben wir das noch nie stillschweigend getan.

Ich denke, was hier vor sich geht, ist, dass das Modal für das Feature, das wir hinzugefügt haben, es so klingen lässt, als sollte dies auch unterdrückt werden. Ich denke, wir haben die Erwartungen mit diesem Modal verschoben und erfüllen sie nun nicht mehr.

Das ist zumindest meine Vermutung.

Ich denke, dies ist eine legitime #feature- oder #ux-Anfrage, und ich unterstütze sie voll und ganz. Aber ich glaube nicht, dass es sich technisch um einen Fehler oder eine Regression handelt.

2 „Gefällt mir“

(Beitrag vom Autor gelöscht)

Ich dachte, dies sei Teil des Zwecks, dies als Option in Massenaktionen einzubeziehen (um das Verhalten von „Kategorie-Bearbeitungsbenachrichtigungen deaktivieren“ und „Tag-Bearbeitungsbenachrichtigungen deaktivieren“ bei Bedarf anzupassen).

Ohne dies wäre die Funktion ziemlich eingeschränkt.

2 „Gefällt mir“

Guter Punkt. Ich bin ehrlich gesagt nicht sicher.

Ich sehe, dass dies genau der Anwendungsfall ist, der in der ursprünglichen Funktionsanfrage beschrieben wurde:

Ich sehe auch, dass mehrere Leute dasselbe dachten, als sie die neue Checkbox sahen: Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

Auch die PR selbst lässt vermuten, dass dies die Absicht war:

Wenn das Kontrollkästchen „Diese Aktion stillschweigend ausführen“ aktiviert wurde, sollte der Sidekiq-Job :notify_category_change nicht in die Warteschlange gestellt werden.

Aber ich sehe keine explizite Spezifikationsabdeckung für diesen Fall.

Zugegebenermaßen bin ich etwas aus der Übung, Ruby-Spezifikationen vorzubereiten, da ich in den letzten Jahren nicht mehr täglich Code geschrieben habe, aber dies scheint eine mögliche Lücke zu sein.

Es riecht für mich eher nach einem Fehler. Ich bin mir nicht sicher, ob es eine Regression ist oder nicht, aber es fühlt sich an, als ob uns die Abdeckung für diesen Fall fehlt.

3 „Gefällt mir“

Ahah! Ich glaube, ich habe das Problem gefunden :bug:

Die Option @silent wurde nicht ordnungsgemäß weitergegeben, was dazu führte, dass Benutzer, die die Ziel-Kategorie beobachteten, Benachrichtigungen erhielten, selbst wenn das Kontrollkästchen „Diese Aktion stillschweigend ausführen“ aktiviert war.

6 „Gefällt mir“

Großartig! Und vielen Dank, dass Sie das ernst genommen haben und für die super schnelle Lösung! Ich habe es gerade getestet und alles scheint gut zu sein.

Ich wollte nur anmerken, dass der Text in der Benutzeroberfläche jetzt anders/umgekehrt ist – sodass das Anklicken des Kontrollkästchens genau das Gegenteil von dem bewirkt, was es früher getan hat. Aber dieser Teil scheint einwandfrei zu funktionieren, also hoffentlich alles gut.

Früher war es:

Jetzt ist es:

1 „Gefällt mir“

:telephone: Bitte hören Sie genau zu, da sich unsere Menüoptionen geändert haben…

2 „Gefällt mir“