Dejé esa opción sin marcar, ¡y les envió correos electrónicos como locos! La categoría de destino está configurada como Watching first post (Siguiendo la primera publicación), pero antes de este cambio, esas notificaciones se habrían suprimido.
@zogstrip No veo cobertura de pruebas para este caso.
Según mi lectura, las especificaciones de describe "silent option" solo cubren los escenarios en los que el tema que se está moviendo ya está siendo observado.
El caso que se describe aquí es para el escenario en el que la categoría de destino está “observada” o “observando la primera publicación”.
@nathank, hasta donde sé, nunca hemos hecho eso en silencio.
Creo que lo que está pasando aquí es que la ventana modal para la función que sí agregamos hace que suene como si eso también debiera silenciarse. Creo que hemos cambiado las expectativas con esta ventana modal y ahora no las estamos cumpliendo.
Esa es mi corazonada de todos modos.
Creo que esta es una solicitud legítima de Feature o UX, y la apoyo totalmente. Pero no creo que sea técnicamente un error o una regresión.
Pensé que este era parte del propósito de incluir esto como una opción en las acciones masivas (para que coincidiera con el comportamiento de Deshabilitar notificaciones de edición de categoría y Deshabilitar notificaciones de edición de etiqueta de forma puntual).
Sin eso, la característica sería bastante limitada.
La PR en sí también da a entender que esa era la intención:
Cuando se haya marcado la casilla “Realizar esta acción en silencio”, no se debe poner en cola el trabajo :notify_category_change sidekiq.
Pero no veo una cobertura de especificación explícita para este caso.
Es cierto que estoy un poco fuera de práctica leyendo especificaciones ruby, ya que no he escrito código a diario durante los últimos años, pero esto me parece una posible laguna.
Ahora me huele más a un error. No estoy seguro de si es una regresión o no, pero siento que nos falta cobertura para este caso.
La opción @silent no se estaba propagando correctamente, lo que provocaba que los usuarios que seguían la categoría de destino recibieran notificaciones incluso cuando la casilla “Realizar esta acción en silencio” estaba marcada.
¡Genial! Y gracias por tomarse esto en serio y por la solución súper rápida. Acabo de probarlo y todo parece estar bien.
Solo quería señalar que el texto en la interfaz de usuario ahora es diferente/invertido, de modo que marcar la casilla hace exactamente lo contrario de lo que solía hacer. Pero esa parte parece estar funcionando bien, así que espero que todo esté bien.
