Je n’ai pas coché cette option, et cela leur a envoyé des e-mails à tour de bras ! La catégorie de destination est configurée sur Suivre le premier message, mais avant ce changement, ceux-ci auraient été supprimés.
@zogstrip Je ne vois pas de couverture de test pour ce cas.
D’après ma lecture, les spécifications describe "silent option" ne couvrent que les scénarios où le sujet déplacé est déjà surveillé.
Le cas décrit ici concerne le scénario où la catégorie de destination est “surveillée” ou “surveille le premier message”.
@nathank, à ma connaissance, nous n’avons jamais fait cela silencieusement.
Je pense que ce qui se passe ici, c’est que la modale pour la fonctionnalité que nous avons ajoutée donne l’impression que cela devrait également être masqué. Je pense que nous avons modifié les attentes avec cette modale et que nous ne les satisfaisons plus maintenant.
C’est mon intuition en tout cas.
Je pense que c’est une demande légitime de #fonctionnalité ou d’ UX, et je soutiens entièrement son implémentation. Mais je ne pense pas que ce soit techniquement un bogue ou une régression.
Je pensais que cela faisait partie de l’objectif d’inclure ceci comme option dans les actions groupées (pour correspondre au comportement de Désactiver les notifications de modification de catégorie et Désactiver les notifications de modification d'étiquette de manière ponctuelle).
Sans cela, la fonctionnalité serait plutôt limitée.
Le PR lui-même semble également indiquer que c’était l’intention :
Lorsque la case à cocher « Effectuer cette action en silence » a été cochée, le travail Sidekiq :notify_category_change ne doit pas être mis en file d’attente.
Mais je ne vois pas de couverture de spécification explicite pour ce cas.
Je dois admettre que je suis un peu rouillé pour préparer les spécifications Ruby puisque je n’écris plus de code au quotidien depuis quelques années, mais cela ressemble à une lacune possible.
Cela sent plus un bug maintenant. Je ne sais pas s’il s’agit d’une régression ou non, mais j’ai l’impression qu’il nous manque une couverture pour ce cas.
L’option @silent ne se propageait pas correctement, ce qui faisait que les utilisateurs surveillant la catégorie de destination recevaient des notifications même lorsque la case à cocher « Effectuer cette action en silence » était cochée.
Brillant ! Et merci à vous deux de prendre cela au sérieux et pour la correction super rapide ! Je viens de tester et tout semble bon.
Je voulais juste noter que le texte dans l’interface utilisateur est maintenant différent / inversé - de sorte que cocher la case fait exactement le contraire de ce qu’elle faisait auparavant. Mais cet aspect semble fonctionner correctement, donc j’espère que tout va bien.