Skip to content
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

Ads on/off toggle for different profiles causes confusion when using profile with Ads off #7441

Closed
LaurenWags opened this issue Dec 17, 2019 · 5 comments · Fixed by brave/brave-core#5574

Comments

@LaurenWags
Copy link
Member

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

  1. Clean profile (Profile A).
  2. Enable Rewards. Now disable Rewards. Leave window for this profile open.
  3. Add another profile (Profile B).
  4. Enable Rewards, set ads to show 5x per hour.
  5. Visit a site like brave.com in Profile B window and then after page loads, close the window for Profile B.
  6. Browse normally on window for Profile A.

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)

Brave 1.1.22 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? probably
  • Can you reproduce this issue with the dev channel? probably
  • Can you reproduce this issue with the nightly channel? probably

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? on all profiles yes.
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

cc @tmancey

@Brave-Matt
Copy link

Another user reporting similar behavior on Community:
https://community.brave.com/t/having-problem-with-2-profiles-at-the-same-time/100756/3

@tmancey tmancey added the priority/P3 The next thing for us to work on. It'll ride the trains. label Jan 8, 2020
@tmancey tmancey added the bug label Jan 20, 2020
@tmancey tmancey removed QA/Test-Plan-Specified QA/Yes priority/P3 The next thing for us to work on. It'll ride the trains. labels Jan 20, 2020
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes labels May 1, 2020
@tmancey tmancey self-assigned this May 16, 2020
@tmancey tmancey added this to the 1.11.x - Nightly milestone May 20, 2020
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 24, 2020

Currently, ads are not shown in 1.11.75. QA blocked until PR brave/brave-core#5751 is uplifted

@GeetaSarvadnya
Copy link

PR #5751 is uplifted to 1.11.x hence removing QA blocked label

@LaurenWags
Copy link
Member Author

LaurenWags commented Jul 7, 2020

Verified using

Brave | 1.11.90 Chromium: 83.0.4103.116 (Official Build) dev (64-bit)
-- | --
Revision | 8f0c18b4dca9b6699eb629be0f51810c24fb6428-refs/branch-heads/4103@{#716}
OS | macOS Version 10.14.6 (Build 18G3020)

Verification passed on

Brave 1.11.94 Chromium: 84.0.4147.85 (Official Build) (64-bit)
Revision e20c13e958edc3707db3d26d513780a7f6331a4b-refs/branch-heads/4147@{# 837}
OS Windows 10 OS Version 1809 (Build 17763.1282)

Verification passed on

Brave 1.11.94 Chromium: 84.0.4147.85 (Official Build) (64-bit)
Revision e20c13e958edc3707db3d26d513780a7f6331a4b-refs/branch-heads/4147@{#837}
OS Ubuntu 18.04 LTS

@LaurenWags
Copy link
Member Author

Note, since issue is not resolved, removed the release-notes/include tag. Please adjust if necessary @rebron @kjozwiak .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
7 participants