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 Panel in 0.60.x should have 'Coming Soon' text #3421

Closed
LaurenWags opened this issue Feb 19, 2019 · 6 comments
Closed

Ads Panel in 0.60.x should have 'Coming Soon' text #3421

LaurenWags opened this issue Feb 19, 2019 · 6 comments

Comments

@LaurenWags
Copy link
Member

Description

As per #3141 Ads panel on 0.60.x Release Channel should reflect 'Coming Soon' as it did with 0.59.x on Release Channel

Steps to Reproduce

Scenario 1

  1. Clean profile on 0.60.44 Release Channel build
  2. Enable Rewards
  3. View Ads Panel

Scenario 2

  1. Clean profile on 0.59.35 Release Channel version
  2. Enable Rewards
  3. Update to 0.60.44
  4. View Ads panel

Actual result:

0.60.44 shows
screen shot 2019-02-19 at 2 00 54 pm

Expected result:

0.59.35
screen shot 2019-02-19 at 1 55 52 pm

Reproduces how often:

Easily

Brave version (brave://version info)

Brave 0.60.44 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? n/a - this needs to be on Release Channel only

Website problems only:

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

Additional Information

cc @mandar-brave @jsecretan

@kjozwiak
Copy link
Member

@LaurenWags ads are still disabled and there's no way of enabling them correct? It's just not using the correct "Coming soon" asset in brave://rewards?

@LaurenWags
Copy link
Member Author

LaurenWags commented Feb 19, 2019

correct @kjozwiak . I launched 0.60.44 with --enable-logging=stderr --vmodule=brave_ads=3 --brave-ads-debug, enabled Rewards (not able to enable Ads), and verified no AdsService messages in terminal. If there's another way I can/should be checking that Ads isn't running in the background that would be good info to have as well.

@kjozwiak
Copy link
Member

@mrose17 @jsecretan is there any other way to make sure that the ad service isn't running other than what @LaurenWags mentioned above?

@mandar-brave @rebron I don't think this is a blocker. As @LaurenWags mentioned above, seems like ads are disabled but the messaging is off under brave://rewards where it doesn't show the "Coming Soon" message. Thoughts?

@mrose17
Copy link
Member

mrose17 commented Feb 19, 2019

that is the only way that i know of.

@tmancey
Copy link
Contributor

tmancey commented Feb 21, 2019

You can also check that files in the Default/ads_service directory are not created or updated when Ads is turned off. i.e. on macOS ~/Library/Application Support/BraveSoftware/Brave-Browser-Development/Default/ads_service

@LaurenWags
Copy link
Member Author

LaurenWags commented Feb 21, 2019

Verified passed with

Brave 0.60.45 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Mac OS X
  • Verified upgraded profiles and clean profiles show the 'Coming soon' text for Ads
    screen shot 2019-02-21 at 5 44 00 pm

Verification passed on

Brave 0.60.45 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Windows 7

Used test run from the description.

image

Verification PASSED on Mint 19.1 x64 using the following build:

Brave 0.60.45 Chromium: 72.0.3626.109 (Official Build) (64-bit)
Revision fae8db7ab9280fa6704a59980263c804f809ebd5-refs/branch-heads/3626@{#857}
OS Linux

screen shot 2019-02-21 at 9 25 15 pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment