Skip to content

fix(notifications): per-filter settings override global#2208

Merged
zze0s merged 3 commits intodevelopfrom
fix/notifications-per-filter-muted-events
Oct 8, 2025
Merged

fix(notifications): per-filter settings override global#2208
zze0s merged 3 commits intodevelopfrom
fix/notifications-per-filter-muted-events

Conversation

@zze0s
Copy link
Copy Markdown
Collaborator

@zze0s zze0s commented Sep 30, 2025

What is the relevant ticket/issue

Reported on Discord

What's this PR do?

Add
  • Muted label in notifications settings panel
Change
  • Adds a new FilterMuted check and updated checks in notification senders CanSendPayload

How should this be manually tested?

  • Add a "global" notification with all events
  • Add a "special" notification with no global events
  • In a filter set the global notification to muted, and the special to selected events

Screenshots (if appropriate)

image image image

Does the documentation or dependencies need an update?

  • no

@zze0s zze0s added this to the v1.67.1 milestone Sep 30, 2025
@zze0s zze0s moved this to Review in Autobrr development Sep 30, 2025
@zze0s zze0s changed the title fix(notifications): skip muted fix(notifications): per-filter notification ovrrides global Oct 8, 2025
@zze0s zze0s changed the title fix(notifications): per-filter notification ovrrides global fix(notifications): per-filter settings override global Oct 8, 2025
@zze0s zze0s merged commit 18d452c into develop Oct 8, 2025
53 of 68 checks passed
@zze0s zze0s deleted the fix/notifications-per-filter-muted-events branch October 8, 2025 14:58
@github-project-automation github-project-automation bot moved this from Review to Done in Autobrr development Oct 8, 2025
thmro pushed a commit to thmro/autobrr that referenced this pull request Feb 21, 2026
* fix(notifications): skip muted

* fix(notifications): per-filter overrides global
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

1 participant