-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Weekly roundup notification despite notifications off #15816
Comments
Hi @adityabhaskar, thanks for bringing this up!
Do you mean that you went to Notifications (bottom menu) > Settings (gear icon) and toggled the toggle there from "On" to "Off"? If so, I would expect the same thing you do. Since all Notification Types (Notifications tab, Email, and App notifications) are listed for individual sites in this Notification Settings section, I would expect toggling the whole thing to "off" to turn off all of those notifications. cc: @Gio2018 this issue would effect the Weekly Roundup, but might also be a discussion around what we expect from that Notifications Settings on/off toggle. |
related: #15318 |
This bug is still active on version 19.4-rc-1. |
With the fix, weekly roundup notifications, create site notifications, and blogging reminder notifications won't be delivered when the Notification Settings switch is off. |
Expected behavior
When all notifications are off, the weekly roundup notifications are still delivered.
Actual behavior
The weekly roundup notifications shouldn't be delivered.
Ideally, there should be a toggle to specifically disable just them. I'd rather have one or two notifications enabled, but toggling 7-8 toggles per site is unnecessarily hard, so I've disabled all notifications at the top of the notifications settings screen. Weekly roundup still came through!
Now I've removed the notification permissions for the app from system settings, but thought I'd let you know just in case this is a bug and not a growth hack.
Steps to reproduce the behaviour
Tested on [device], Android [version], WPAndroid [version]
Pixel 5, Android 12, WP 18.8
The text was updated successfully, but these errors were encountered: