L’ho lasciato deselezionato e li ha bombardati di email! La categoria di destinazione è configurata come Watching first post (Segui primo post), ma prima di questa modifica sarebbero state soppresse.
@zogstrip Non vedo una copertura di test per questo caso.
Secondo la mia lettura, le specifiche describe "silent option" coprono solo gli scenari in cui il topic da spostare è già tracciato.
Il caso descritto qui riguarda invece lo scenario in cui la categoria di destinazione è “tracciata” o “traccia il primo post”.
@nathank, per quanto ne sappia, non l’abbiamo mai fatto in modo silenzioso.
Credo che ciò che sta accadendo sia che il modale per la funzionalità che abbiamo aggiunto fa sembrare che anche questa dovrebbe essere silenziata. Credo che con questo modale abbiamo modificato le aspettative e ora non le stiamo soddisfacendo.
Questa è la mia impressione, comunque.
Penso che si tratti di una richiesta legittima Contribute > Feature o Contribute > UX, e la sostengo pienamente. Ma non penso che sia tecnicamente un bug o un regression.
Pensavo che questo fosse parte dello scopo di includerlo come opzione nelle azioni di massa (per eguagliare il comportamento di Disabilita notifiche di modifica categoria e Disabilita notifiche di modifica tag su base ad hoc).
Senza di essa, la funzionalità sarebbe piuttosto limitata.
Anche la PR stessa sembra indicare che questa fosse l’intenzione:
Quando la casella di controllo “Esegui questa azione in silenzio” è stata spuntata, il job sidekiq :notify_category_change non dovrebbe essere accodato.
Ma non vedo una copertura di specifica esplicita per questo caso.
Devo ammettere che sono un po’ arrugginito nel preparare le specifiche ruby dato che non scrivo codice quotidianamente da qualche anno, ma questo sembra un possibile vuoto.
Mi puzza più di un bug adesso. Non sono sicuro se sia una regressione o meno, ma sembra che ci manchi la copertura per questo caso.
L’opzione @silent non veniva propagata correttamente, il che causava la ricezione di notifiche da parte degli utenti che seguivano la categoria di destinazione anche quando la casella “Esegui questa azione in silenzio” era selezionata.
Brillante! E grazie sia per aver preso sul serio la cosa sia per la correzione super veloce! L’ho appena testato e sembra tutto a posto.
Volevo solo notare che il testo nell’interfaccia utente è ora diverso / invertito, in modo che spuntare la casella faccia esattamente l’opposto di quello che faceva prima. Ma quell’aspetto sembra funzionare bene, quindi spero che sia tutto a posto.