-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Comments
Reproducible on macOS.
Noticed that 0.69.128 is a minor chromium version behind 0.68.140 - unsure if that could be causing an issue? |
Reproduces with latest 0.68.x (0.68.141) and upgrading to 0.69.129.
|
The issue is still reproducible in
|
When QA verifies this issue, we should also check the following scenario: |
@LaurenWags @GeetaSarvadnya just to note this is still waiting uplift approval |
@ryanml do you believe your fix will also resolve the steps I outlined in: #6148 (comment) ? |
@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 |
Verified passed with
Verification passed on
Verification passed on
|
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)
Also verify scenario from #6148 (comment)
Actual result:
Ads panel disappeared from brave://rewards page after upgrade (0.68.140 to 0.69.128)
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)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc: @tmancey @jsecretan @rebron @brave/legacy_qa
The text was updated successfully, but these errors were encountered: