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.
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.
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
Haben Sie vielleicht eine Möglichkeit, Reproduktionsschritte zu finden?
@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.
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.
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.
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.
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.