-
Notifications
You must be signed in to change notification settings - Fork 221
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
feat: resetting the timeline when an ignore user list event is received #1705
feat: resetting the timeline when an ignore user list event is received #1705
Conversation
Co-authored-by: Jonas Platte <jplatte@matrix.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two more nitpicks, otherwise looks good!
Co-authored-by: Jonas Platte <jplatte@matrix.org>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1705 +/- ##
==========================================
- Coverage 76.13% 76.12% -0.02%
==========================================
Files 137 137
Lines 15234 15241 +7
==========================================
+ Hits 11599 11602 +3
- Misses 3635 3639 +4
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
# Conflicts: # bindings/matrix-sdk-ffi/src/sliding_sync.rs # crates/matrix-sdk-base/src/client.rs # crates/matrix-sdk/src/client/mod.rs
I see that some changes have been made on how we handle the reception of the tasks, so I aligned my branch to these changes, let me know if they work for you @jplatte |
This will require also a restart of SS and the cache to be cleaned, which @Hywan is working on separately, otherwise in offline cases the cold cache could restore the old messages.