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

[Desktop] re-enabling Rewards from NTP widget doesn't update display of widget #11039

Closed
LaurenWags opened this issue Jul 31, 2020 · 2 comments · Fixed by brave/brave-core#6321

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Jul 31, 2020

Description

Found while testing #9526

If you try to re-enable rewards from the Rewards NTP widget it doesn't look like it worked, but if you go to brave://rewards page you can see that Rewards are toggled on. Additionally, BAT Ads Service and BAT Ledger Service are running in task manager. A page refresh fixes the display of the widget.

Steps to Reproduce

  1. Clean profile 1.12.x, open Task Manager. See that BAT Ads and Ledger services are not running.
  2. Enable Rewards (I did this from panel). See BAT Ads and Ledger Services are now running.
  3. Open a new tab, navigate to brave://rewards.
  4. Toggle Rewards off. See BAT Ads and Ledger Services are no longer running.
  5. Close brave://rewards page.
  6. Open NTP
  7. Click Turn on Rewards button (may need to dismiss SI message depending on which NTP image is being shown to you)

Actual result:

Widget turns from purple (not enabled) to black (enabled) but still displays the messaging to re-join. Services (BAT Ads and BAT Ledger) are running as expected. Page refresh or clicking on Try Again fixes this. Note - Rewards is actually enabled, this seems like a display issue.

Screen Shot 2020-07-31 at 9 52 56 AM

Expected result:

Display should be updated without a page refresh

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.12.105 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS macOS Version 10.14.6 (Build 18G3020)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes for 1.12.x rc
  • Can you reproduce this issue with the beta channel? yes for 1.13.x
  • Can you reproduce this issue with the nightly channel? yes for 1.14.x

Other Additional Information:

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

Miscellaneous Information:

cc @NejcZdovc @rebron @brave/legacy_qa

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jul 31, 2020

Reproduced in Windows 10 x64 - 1.12.105

Reproduced on Ubuntu 18.04

Brave 1.12.105 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS Ubuntu 18.04 LTS

@LaurenWags
Copy link
Member Author

LaurenWags commented Aug 6, 2020

Verified passed with

Brave | 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
-- | --
Revision | a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS | macOS Version 10.14.6 (Build 18G3020)
  • Verified STR from description

Screen Shot 2020-08-06 at 1 49 14 PM

  • Generally confirmed I was able to turn on Rewards from brave://rewards, panel, NTP widget
  • Generally confirmed after turning off Rewards via brave://rewards page, I could re-enable via brave://rewards, panel, NTP
  • Generally confirmed after reset of Rewards I could re-join via brave://rewards, panel, NTP widget

Verification passed on

Brave | 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
-- | --
Revision | a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS | Windows 10 OS Version 1903 (Build 18362.959)

  • Verified STR from description
    image

  • Generally confirmed I was able to turn on Rewards from brave://rewards, panel, NTP widget

  • Generally confirmed after turning off Rewards via brave://rewards page, I could re-enable via brave://rewards, panel, NTP

  • Generally confirmed after reset of Rewards I could re-join via brave://rewards, panel, NTP widget


Verification passed on

Brave 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
Revision a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS Ubuntu 18.04 LTS
  • Verified STR from description
    image

  • Generally confirmed I was able to turn on Rewards from brave://rewards, panel, NTP widget

  • Generally confirmed after turning off Rewards via brave://rewards page, I could re-enable via brave://rewards, panel, NTP

  • Generally confirmed after reset of Rewards I could re-join via brave://rewards, panel, NTP widget

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