Skip to content

MSC4308: Thread Subscriptions extension to Sliding Sync tests #802

Merged
reivilibre merged 9 commits into
mainfrom
rei/ssext_threadsubs
Sep 11, 2025
Merged

MSC4308: Thread Subscriptions extension to Sliding Sync tests #802
reivilibre merged 9 commits into
mainfrom
rei/ssext_threadsubs

Conversation

@reivilibre
Copy link
Copy Markdown
Contributor

@reivilibre reivilibre commented Sep 5, 2025

Closes: element-hq/synapse#18711

Pull Request Checklist

Signed-off-by: Olivier 'reivilibre' oliverw@matrix.org

@reivilibre
Copy link
Copy Markdown
Contributor Author

CI failing because would also need element-hq/synapse#18819 merged on Synapse repo

Comment thread tests/msc4306/thread_subscriptions_sliding_sync_test.go Outdated
return newPos, extension
}

func TestThreadSubscriptionsSlidingSync(t *testing.T) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even though we're organizing this in MSC4306 for convenience and grouping, we should probably call this out as MSC4308 in the comments at-least

@reivilibre reivilibre merged commit 80330aa into main Sep 11, 2025
4 checks passed
@reivilibre reivilibre deleted the rei/ssext_threadsubs branch September 11, 2025 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Threads 2.0: Complement tests

2 participants