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 disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128) #6148

Closed
GeetaSarvadnya opened this issue Sep 24, 2019 · 9 comments · Fixed by brave/brave-core#3536

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 24, 2019

Description

Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128)

Steps to Reproduce

Prerequisites: VPN is connected to the United States and language is set to English (US)

  1. Clean profile 0.68.140
  2. Enable rewards
  3. Make sure ads switch is enabled by default
  4. Disable Ads switch
  5. Upgrade to 0.69.128
  6. Navigate to brave://rewards page and observe Ads panel is completely removed

Also verify scenario from #6148 (comment)

Actual result:

Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128)

Ads panel disappered

image

Expected result:

Ads should panel should not be removed and it should be in disabled state after upgrade

Reproduces how often:

Always

Brave version (brave://version info)

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @tmancey @jsecretan @rebron @brave/legacy_qa

@LaurenWags
Copy link
Member

Reproducible on macOS.

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS macOS Version 10.13.6 (Build 17G5019)

Noticed that 0.69.128 is a minor chromium version behind 0.68.140 - unsure if that could be causing an issue?

@LaurenWags
Copy link
Member

Reproduces with latest 0.68.x (0.68.141) and upgrading to 0.69.129.

Brave 0.69.129 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)

@ryanml ryanml added this to the 0.69.x - Release milestone Sep 25, 2019
@ryanml ryanml self-assigned this Sep 25, 2019
ryanml added a commit to brave/brave-core that referenced this issue Sep 26, 2019
@ryanml ryanml moved this from New Issues to Review/QA in Rewards Sep 26, 2019
ryanml added a commit to brave/brave-core that referenced this issue Sep 26, 2019
ryanml added a commit to brave/brave-core that referenced this issue Sep 26, 2019
@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Sep 26, 2019

Issue persists in one more scenario, open any old profile where ads are Cashed in/ Viewed. Ads panel is removed.

image

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Sep 30, 2019

The issue is still reproducible in 0.69.130

Brave 0.69.130 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 10 OS Version 1803 (Build 17134.1006)

@LaurenWags
Copy link
Member

When QA verifies this issue, we should also check the following scenario:
Have a profile with Rewards On in a region where Ads are not enabled.
Upgrade to version with the fix.
Verify Ads panel is displayed with the "Sorry" message.

@ryanml
Copy link
Contributor

ryanml commented Sep 30, 2019

@LaurenWags @GeetaSarvadnya just to note this is still waiting uplift approval

@LaurenWags
Copy link
Member

@ryanml do you believe your fix will also resolve the steps I outlined in: #6148 (comment) ?

@ryanml
Copy link
Contributor

ryanml commented Sep 30, 2019

@LaurenWags I believe so yes, what my fix resolved was an explicit code return that would not show the panel, so with that gone the panel should continue to function as expected, with state for non-supported region showing appropriately

@NejcZdovc NejcZdovc moved this from Waiting uplift to Done in Rewards Oct 1, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Oct 1, 2019

Verified passed with

Brave 0.69.131 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.13.6 (Build 17G5019)
  • Verified original STR from description. After update to 0.69.131 Ad panel is shown with Ads off as expected.
  • Verified scenario from Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128) #6148 (comment) . After update to 0.69.131 Ad panel is shown with 'Sorry' message as expected. Verified using Japan region.
  • Verified on a clean install of 0.69.131, when Rewards are enabled the Ads panel is shown. Ads increment on the panel as expected.
  • Verified when Ads are on and have been viewed with 0.68.142, after update to 0.69.131 Ad panel is shown with Ads on as expected. Data is retained after upgrade. Verified Ads panel increments when ads are shown after upgrade.

Verification passed on

Brave 0.69.131 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified original STR from description. After update to 0.69.131 Ad panel is shown with Ads off as expected.
  • Verified scenario from Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128) #6148 (comment) . After update to 0.69.131 Ad panel is shown with 'Sorry' message as expected. Verified using the India region.
  • Verified on a clean install of 0.69.131, when Rewards are enabled the Ads panel is shown. Ads increment on the panel as expected.
  • Verified when Ads are on and have been viewed with 0.68.142, after update to 0.69.131 Ad panel is shown with Ads on as expected. Data is retained after upgrade. Verified Ads panel increments when ads are shown after the upgrade.

Verification passed on

Brave 0.69.131 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS Ubuntu 18.04 LTS
  • Verified original STR from description. After update to 0.69.131 Ad panel is shown with Ads off as expected.
  • Verified scenario from Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128) #6148 (comment) . After update to 0.69.131 Ad panel is shown with 'Sorry' message as expected. Verified using the India region.
  • Verified on a clean install of 0.69.131, when Rewards are enabled the Ads panel is shown. Ads increment on the panel as expected.
  • Verified when Ads are on and have been viewed with 0.68.142, after update to 0.69.131 Ad panel is shown with Ads on as expected. Data is retained after upgrade. Verified Ads panel increments when ads are shown after the upgrade.

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