mirror/matrix-rust-sdk
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-rust-sdk.git synced 2026-05-10 08:53:00 -04:00
Code Issues Packages Projects Releases Wiki Activity

notification client(bugfix): don't filter out the notification if we couldn't compute push actions with /context

Browse Source 
This is in line with what the other method using sliding sync does. This
wasn't tested before, because this required `filter_by_push_rules()` to
be enabled in the notification client; now that it's the default, the
test revealed the bug, and so it could be fixed.
This commit is contained in:
Benjamin Bouvier
2024-05-01 13:09:01 +02:00
parent 0ba4e42161
commit f69db1d169
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
crates/matrix-sdk-ui/src/notification_client.rs
View File

@@ -494,12 +494,10 @@ impl NotificationClient {
timeline_event = decrypted_event;
}
if !timeline_event
.push_actions
.as_ref()
.is_some_and(|actions| actions.iter().any(|a| a.should_notify()))
{
return Ok(None);
if let Some(actions) = timeline_event.push_actions.as_ref() {
if !actions.iter().any(|a| a.should_notify()) {
return Ok(None);
}
}
Ok(Some(