Skip to content

Fix bug where sliding sync ignored room_id_to_include option#18535

Merged
erikjohnston merged 10 commits into
developfrom
erikj/ss_exclude_rooms_from_sync
Jun 13, 2025
Merged

Fix bug where sliding sync ignored room_id_to_include option#18535
erikjohnston merged 10 commits into
developfrom
erikj/ss_exclude_rooms_from_sync

Conversation

@erikjohnston

Copy link
Copy Markdown
Member

This was correctly handled for the "fallback" case where the background updates hadn't finished

This was correctly handled for the "fallback" case where the background
updates hadn't finished.
@erikjohnston erikjohnston marked this pull request as ready for review June 10, 2025 09:53
@erikjohnston erikjohnston requested a review from a team as a code owner June 10, 2025 09:53
Comment thread tests/rest/client/sliding_sync/test_sliding_sync.py Outdated
Comment thread tests/rest/client/sliding_sync/test_sliding_sync.py Outdated
Comment thread synapse/handlers/sliding_sync/room_lists.py Outdated
erikjohnston and others added 2 commits June 11, 2025 15:07
Co-authored-by: Eric Eastwood <erice@element.io>
So that we don't include them in the list ops.
Comment thread synapse/handlers/sliding_sync/room_lists.py
Comment thread tests/rest/client/sliding_sync/test_sliding_sync.py Outdated
@erikjohnston erikjohnston force-pushed the erikj/ss_exclude_rooms_from_sync branch from 8d34210 to 04d2750 Compare June 12, 2025 13:05
Comment thread synapse/handlers/sliding_sync/room_lists.py
Comment thread synapse/util/__init__.py Outdated
@erikjohnston erikjohnston force-pushed the erikj/ss_exclude_rooms_from_sync branch 3 times, most recently from 3d610f6 to 0f46209 Compare June 13, 2025 09:55
@erikjohnston erikjohnston force-pushed the erikj/ss_exclude_rooms_from_sync branch from 0f46209 to e9baf1f Compare June 13, 2025 09:56
@erikjohnston erikjohnston merged commit 1709957 into develop Jun 13, 2025
43 checks passed
@erikjohnston erikjohnston deleted the erikj/ss_exclude_rooms_from_sync branch June 13, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants