Inondation involontaire d'emails / notifications lors de la tentative de déplacer silencieusement des Topics entre catégories

Je viens d’avoir un problème en déplaçant 30 sujets entre catégories à cause d’une tempête d’e-mails envoyée à mes utilisateurs les plus précieux.

J’ai remarqué qu’après la dernière mise à jour, le texte avait changé pour la fonctionnalité qui permet de supprimer les notifications lorsqu’un groupe de sujets sélectionnés en bloc voit sa catégorie modifiée. Il affiche maintenant ceci :

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.

Que se passe-t-il ici ?

5 « J'aime »

Désolé pour cela @nathank :frowning:

J’ai essayé de reproduire localement et j’ai amélioré notre couverture de spécifications pour nous assurer que nous gérions ce cas…

… mais tout semble fonctionner comme prévu :thinking:

Auriez-vous une chance de trouver des étapes de reproduction ?

1 « J'aime »

@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.

2 « J'aime »

(message supprimé par l’auteur)

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.

2 « J'aime »

Bon point. Je n’en suis honnêtement pas sûr.

Je vois que c’est exactement le cas d’utilisation décrit dans la demande de fonctionnalité initiale :

Je vois aussi que plusieurs personnes ont pensé la même chose lorsqu’elles ont vu la nouvelle case à cocher : Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

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.

3 « J'aime »

Ahah ! Je pense avoir trouvé le problème :bug:

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.

6 « J'aime »

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.

Avant c’était :

Maintenant c’est :

1 « J'aime »

:telephone: Veuillez écouter attentivement, car nos options de menu ont changé…

2 « J'aime »