Skip to content

Add spaces filtering to Sliding Sync /sync #17280

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 19 commits into
base: develop
Choose a base branch
from
Draft

Add spaces filtering to Sliding Sync /sync #17280

wants to merge 19 commits into from

Conversation

MadLittleMods
Copy link
Contributor

@MadLittleMods MadLittleMods commented Jun 6, 2024 •
edited
Loading

Add spaces filtering to Sliding Sync /sync

Based on MSC3575: Sliding Sync

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
    • Feel free to credit yourself, by adding a sentence "Contributed by @github_username." or "Contributed by [Your Name]." to the end of the entry.
  • Code style is correct
    (run the linters)

@MadLittleMods MadLittleMods mentioned this pull request Jun 6, 2024
Closed
3 tasks
@MadLittleMods
Update changelog number
20ee328
@MadLittleMods
Reference actual filter code
dd43938
@MadLittleMods
Condense true/false tests
88fe201
@MadLittleMods
Merge branch 'madlittlemods/msc3575-sliding-sync-filter-dms' into mad…
0c0a2b9 
…littlemods/msc3575-sliding-sync-filter-spaces2

Conflicts:
	tests/handlers/test_sliding_sync.py
@MadLittleMods
Less test bulk
48eca7d
@MadLittleMods
Incorporate to_token to filters
7aa0519
@MadLittleMods
Explain why no to_token for global account data
a6e5798
@MadLittleMods
Add docstring
5dd6d37
@MadLittleMods
Merge branch 'develop' into madlittlemods/msc3575-sliding-sync-filter…
d0d198f 
…-dms

Conflicts:
	synapse/handlers/sliding_sync.py
@MadLittleMods
Remove import workaround
271ae6f
@MadLittleMods
Remove sneaky log
f69d1c5
@MadLittleMods
Merge branch 'develop' into madlittlemods/msc3575-sliding-sync-filter…
9896478 
…-dms
@MadLittleMods
Merge branch 'develop' into madlittlemods/msc3575-sliding-sync-filter…
eaaf408 
…-dms
Base automatically changed from madlittlemods/msc3575-sliding-sync-filter-dms to develop June 13, 2024 18:56
@MadLittleMods
Merge branch 'madlittlemods/msc3575-sliding-sync-filter-dms' into mad…
5de7d56 
…littlemods/msc3575-sliding-sync-filter-spaces2

Conflicts:
	synapse/handlers/sliding_sync.py
@MadLittleMods
Merge branch 'develop' into madlittlemods/msc3575-sliding-sync-filter…
8b42e2a 
…-spaces2

Conflicts:
	synapse/handlers/sliding_sync.py
	tests/handlers/test_sliding_sync.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
A-Sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MadLittleMods