-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Ads on/off toggle for different profiles causes confusion when using profile with Ads off #7441
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
Labels
bug
feature/ads
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/exclude
Milestone
Comments
Another user reporting similar behavior on Community: |
32 tasks
Currently, ads are not shown in 1.11.75. QA blocked until PR brave/brave-core#5751 is uplifted |
PR #5751 is uplifted to 1.11.x hence removing QA blocked label |
Verified using
Verification passed on
Verification passed on
|
This was referenced Jul 8, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
feature/ads
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/exclude
Description
Originally reported here: https://www.reddit.com/r/brave_browser/comments/ec0xfh/ads_are_off_still_seeing_brave_ads/
@Miyayes surfaced the above thread. While discussing it and attempting to reproduce, it was determined that if a user has a profile with ads/rewards on and one with ads/rewards off, even if a window is not open for the profile with ads on, ad notifications are still presented to the user. Per internal discussions we probably want to limit this to only show ads when there is a window open for the profile which is ads enabled. cc @jsecretan @rebron @mandar-brave @kjozwiak who were also part of the discussion.
Steps to Reproduce
Actual result:
You will be served ad notifications.
Expected result:
Per discussion, we should respect the profile you are using and not serve ad notifications.
Reproduces how often:
easily
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @tmancey
The text was updated successfully, but these errors were encountered: