Les liens vers les messages cachés ne suppriment pas les filtres

Je ne sais pas s’il s’agit d’un seul bug ou de deux, mais voici les étapes pour les reproduire. J’ai dû le faire ici plutôt que sur try.discourse.org car les réponses filtrées n’y sont pas activées.

  1. Allez au message n°6 dans ce fil, faites défiler jusqu’en bas et cliquez pour voir ses 6 réponses :

  2. Faites défiler jusqu’au dernier message (n°25, qui est une réponse directe au n°6) et cliquez sur son lien vers le message n°24 :

Comportement attendu : le message n°24 devient visible (les filtres peuvent être supprimés ou non)

Comportement réel : la page fait défiler vers le haut et vers le bas, mais le n°24 n’apparaît jamais

Passons au bug n°2 :

  1. Cliquez sur « Voir 6 réponses masquées »

  2. Faites défiler vers le bas jusqu’à ce que vous voyiez le message n°24 et cliquez sur son lien vers le message n°8

Comportement attendu : le n°8 apparaît

Comportement réel : le n°7 apparaît

1 « J'aime »

@pmusaraj Si vous avez le temps, pourriez-vous confirmer s’il s’agit bien d’un (ou deux) bug(s) ou s’il s’agit du comportement prévu ? Peut-être que je interprète mal quelque chose.

Oui, cela semble être le même bogue. C’est un cas limite important ; il est assez rare de lier des réponses comme dans l’exemple, mais je vais l’ajouter à ma liste de tâches.

1 « J'aime »

Un cas plus courant est lorsque le lien est en fait une notification de @mention

Hmm, je ne suis pas sûr de suivre : comment un lien de mention pourrait-il pointer vers une réponse dans le même flux de messages ?

J’ai utilisé un lien pour rendre l’exemple reproductible, mais cela a en réalité été signalé par un utilisateur qui voyait une réponse filtrée et a reçu une mention sur le même sujet. Lorsqu’il a cliqué sur la notification, il a obtenu le comportement que j’ai signalé.

2 « J'aime »

C’est corrigé maintenant @Nacho_Caballero via DEV: Clear filters when navigating to a post (#12557) · discourse/discourse@524890c · GitHub

3 « J'aime »

Cela fonctionne parfaitement ! Merci, Penar.

Cela a nécessité étonnamment peu de code pour être corrigé, mais je ne pense pas que j’aurais pu le résoudre seul. :sweat_smile:

1 « J'aime »

Ce sujet a été automatiquement fermé après 16 heures. De nouvelles réponses ne sont plus autorisées.