Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Threads filter does the opposite of what you select #23999

Closed
robintown opened this issue Dec 14, 2022 · 8 comments
Closed

Threads filter does the opposite of what you select #23999

robintown opened this issue Dec 14, 2022 · 8 comments
Labels
A-Threads O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@robintown
Copy link
Member

Steps to reproduce

  1. Open the threads list
  2. Select 'my threads' from the dropdown
  3. Observe that the list still shows all threads
  4. Select 'all threads' from the dropdown
  5. Observe that the list now shows only threads you have participated in

Outcome

What did you expect?

'My threads' should show my threads, and 'all threads' should show all threads

What happened instead?

'My threads' shows all threads, and 'all threads' shows my threads

Operating system

NixOS unstable

Browser information

Firefox 107.0.1

URL for webapp

develop.element.io

Application version

Element version: a7124c9-react-8869374c6bfb-js-447319737a05 Olm version: 3.2.12

Homeserver

Synapse 1.72.0

Will you send logs?

No

@robintown robintown added T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely A-Threads labels Dec 14, 2022
@justjanne
Copy link
Contributor

This should have been fixed by matrix-org/matrix-react-sdk#9746

@t3chguy
Copy link
Member

t3chguy commented Dec 19, 2022

@justjanne 8869374c6bfb is 4 days old, your change landed 5 days ago - I'm inclined to mark this as a release blocker

@t3chguy
Copy link
Member

t3chguy commented Dec 19, 2022

@justjanne could #24040 be considered at the same time, alone probably not a release blocker but definitely backport-worthy

@daniellekirkwood
Copy link
Contributor

While I'm sure this is confusing to users I don't believe it should be a release blocker - It's marked as Occasional and Minor which makes it a P3 issue. Also - users still have access to all their threads in the list (and many users are unaware of the drop down filter).

It would however be great to get this fixed so if not in this release, the next would be ideal 👍

@t3chguy
Copy link
Member

t3chguy commented Dec 19, 2022

@daniellekirkwood the issue is it had a PR which claimed to fix it, which does not inspire confidence in the code quality and lack of tests. I doubt it deserves Occasional if we're pushing threads on-by-default - it is front and centre top of the thread list

The next release is also just shy of a month away, this would make Threads just feel like it didn't go through any QA out of the box for anyone hitting it.

@germain-gg
Copy link
Contributor

@t3chguy the issue happens sporadically. There's ways to end up in that state, but it works most of the time! I'm investigating the issue, but it's more uncommon that occasional IMO

@germain-gg germain-gg added O-Uncommon Most users are unlikely to come across this or unexpected workflow and removed O-Occasional Affects or can be seen by some users regularly or most users rarely labels Dec 20, 2022
@germain-gg
Copy link
Contributor

As discussed with @t3chguy it does most of the time the right thing but can end up in the state described in this ticket.
This is not a release blocker but should absolutely be fixed promptly.

@t3chguy
Copy link
Member

t3chguy commented Jul 19, 2023

I can no longer reproduce this

@t3chguy t3chguy closed this as completed Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Threads O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

6 participants