Links zu versteckten Nachrichten löschen keine Filter

[Nacho_Caballero]

Ich weiß nicht, ob es sich um einen oder zwei Fehler handelt, aber hier sind die Schritte zur Reproduktion. Ich musste dies hier tun, anstatt auf try.discourse.org, da gefilterte Antworten dort nicht aktiviert sind.

1. Gehen Sie zu Beitrag #6 in diesem Thread, scrollen Sie nach unten und klicken Sie, um die 6 Antworten anzuzeigen:

   

2. Scrollen Sie zum letzten Beitrag (Nummer #25, eine direkte Antwort auf #6) und klicken Sie auf den Link zu Beitrag #24:
   

   

Erwartetes Verhalten: Beitrag #24 wird sichtbar (Filter werden möglicherweise entfernt oder nicht)

Tatsächliches Verhalten: Die Seite scrollt hoch und runter, aber #24 erscheint nie

Weiter zu Fehler #2:

3. Klicken Sie auf „6 ausgeblendete Antworten anzeigen"

   

4. Scrollen Sie nach unten, bis Sie Beitrag #24 sehen, und klicken Sie auf den Link zu Beitrag #8

   

Erwartetes Verhalten: #8 erscheint

Tatsächliches Verhalten: #7 erscheint

[Nacho_Caballero]

@pmusaraj Wenn du Zeit hast, könntest du bitte bestätigen, ob es sich hier tatsächlich um einen (oder zwei) Fehler handelt oder ob das beabsichtigtes Verhalten ist? Vielleicht interpretiere ich etwas falsch.

[pmusaraj (Penar Musaraj)]

Ja, das sieht nach ein und demselben Fehler aus. Es ist ein beachtlicher Randfall – es kommt ziemlich selten vor, dass man auf Antworten wie im Beispiel verlinkt, aber ich werde das auf meine To-Do-Liste setzen.

[Nacho_Caballero]

Ein häufiger Fall ist, wenn der Link tatsächlich eine @mention-Benachrichtigung ist.

[pmusaraj (Penar Musaraj)]

Hmm, ich verstehe nicht ganz. Wie könnte ein Verweis auf eine Antwort im selben Post-Stream verlinkt sein?

[Nacho_Caballero]

Ich habe einen Link verwendet, um das Beispiel reproduzierbar zu machen, aber dies wurde tatsächlich von einem Benutzer gemeldet, der eine gefilterte Antwort sah und auf demselben Thema eine Erwähnung erhielt. Als sie auf die Benachrichtigung klickten, trat das von mir gemeldete Verhalten auf.

[pmusaraj (Penar Musaraj)]

Dies ist jetzt behoben @Nacho_Caballero über DEV: Clear filters when navigating to a post (#12557) · discourse/discourse@524890c · GitHub

[Nacho_Caballero]

Das funktioniert perfekt! Danke, Penar.

Das war überraschend wenig Code, um es zu beheben, aber ich glaube nicht, dass ich es allein herausgefunden hätte.