Inundación involuntaria de correos/notificaciones al intentar mover Temas silenciosamente entre categorías

Buen punto. Sinceramente no estoy seguro.

Veo que ese es exactamente el caso de uso descrito en la solicitud de función original:

También veo que varias personas pensaron lo mismo cuando vieron la nueva casilla de verificación: Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

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.

3 Me gusta