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 la copertura dei test per questo caso.
Dalla mia lettura, le specifiche describe "silent option" coprono solo gli scenari in cui l’argomento che viene spostato è già monitorato.
Il caso descritto qui è per lo scenario in cui la categoria di destinazione è “watched” (monitorata) o “watching first post” (monitoraggio primo post).
@nathank, a mia conoscenza, non l’abbiamo mai fatto in silenzio.
Penso che quello che sta succedendo qui sia che la modale per la funzionalità che abbiamo aggiunto la fa sembrare che anche questa dovrebbe essere silenziata. Penso che abbiamo spostato le aspettative con questa modale e ora non le stiamo soddisfacendo.
Questa è la mia impressione, comunque.
Penso che questa sia una legittima richiesta di Feature (funzionalità) o UX (esperienza utente), e la supporto pienamente. Ma non penso che sia tecnicamente un bug o una regressione.
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.
