Brave-browser: Ads Panel in 0.60.x should have 'Coming Soon' text

Created on 19 Feb 2019  路  6Comments  路  Source: brave/brave-browser

Description

As per https://github.com/brave/brave-browser/issues/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

QA Pass-Linux QA Pass-Win64 QA Pass-macOS QTest-Plan-Specified QYes featurads featurrewards release-noteexclude

Most helpful comment

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

All 6 comments

@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?

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.

@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?

that is the only way that i know of.

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

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

Was this page helpful?
0 / 5 - 0 ratings