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

Room ordering has broken in RoomList, with ancient inactive rooms resurfacing as unread #19741

Open
ara4n opened this issue Nov 13, 2021 · 9 comments
Assignees
Labels
A-Room-List O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression Z-GetYourUpdates

Comments

@ara4n
Copy link
Member

ara4n commented Nov 13, 2021

Steps to reproduce

Using today's nightly, with Rooms and People list ordered by activity with 'show rooms with unread messages first' enabled, after about ~30 minutes of use, random ancient rooms appear at the top of the unread messages section of the room sublists, seemingly replacing the correct rooms with unread messages which should be there.

Outcome

Rooms with unread messages are shown in the unread section of the roomlist, ordered by recency - rather than ancient idle rooms appearing there.

Operating system

macOS 12.0.1 on M1

Application version

nightly

How did you install the app?

nightly

Homeserver

matrix.org

Will you send logs?

Yes

@ara4n ara4n added the T-Defect label Nov 13, 2021
@ara4n
Copy link
Member Author

ara4n commented Nov 13, 2021

(rageshake contains more details)

@SimonBrandner SimonBrandner added A-Room-List S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression labels Nov 13, 2021
dbkr added a commit to matrix-org/matrix-react-sdk that referenced this issue Nov 16, 2021
@novocaine novocaine added the O-Uncommon Most users are unlikely to come across this or unexpected workflow label Nov 17, 2021
@novocaine
Copy link
Contributor

@ara4n experienced the behaviour on app.element.io - an old room bubbled up there, not e2e

@ara4n
Copy link
Member Author

ara4n commented Nov 17, 2021

I've chased this down further - it looks to be a regression in element-hq/element-meta#862. The rooms which bubble up are being ordered based on the most recent profile change. However, it looks like it may be intermittent - I saw !XjXlEydwpKIBnirswa:gnugen.ch bubble up from the depths on app.element.io, but not on Nightly. I've also repro'd the bug on Amandine's Nightly (the same old rooms were being resurrected from the depths by Neil changing his nick on Nov 9th), so I think this is probably biting everyone.

@novocaine
Copy link
Contributor

the same old rooms were being resurrected from the depths by Neil changing his nick on Nov 9th

that sounds like #15860 ..?

@ara4n
Copy link
Member Author

ara4n commented Nov 17, 2021

...which was a dup of element-hq/element-meta#862 that nobody spotted. It should have been fixed by matrix-org/matrix-react-sdk#6595, if it hadn't regressed.

@novocaine
Copy link
Contributor

or matrix-org/synapse#9716

@ara4n
Copy link
Member Author

ara4n commented Aug 30, 2022

I just got bitten by this again.

@daniellekirkwood
Copy link
Contributor

@kerryarchibald as we're looking at room list ordering, could you take a look at this and see if it's something straight-forward while we're in that space?

@ara4n
Copy link
Member Author

ara4n commented May 8, 2024

This is still happening, with profile changes causing hundreds of rooms to be marked as unread and rise to the top - drowning out everything else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Room-List O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression Z-GetYourUpdates
Projects
None yet
Development

No branches or pull requests

8 participants