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

"Join Rewards" is being displayed in the rewards panel when restarting Brave #6015

Closed
kjozwiak opened this issue Sep 13, 2019 · 3 comments · Fixed by brave/brave-core#3435
Closed

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Sep 13, 2019

Description

Reported by both @GeetaSarvadnya and @rebron, whenever you restart 0.68.136 Chromium: 77.0.3865.75 the rewards panel will default to Join Rewards even though rewards has been enabled. Because a restart is required when Brave is updated, a good % of our user base has the potential to run into the above.

Steps to Reproduce

Test Case 1

  1. launch 0.68.136 Chromium: 77.0.3865.75
  2. enable rewards via the panel
  3. once enabled/wallet created, restart the browser
  4. click on the reward panel and you'll see "Join Rewards" even though the wallet is created

You can repeat the above several times and you'll notice that the panel keeps reverting back to "Join Rewards"

Test Case 2

  1. launch 0.68.132 Chromium: 76.0.3809.132 (current release)
  2. enable rewards via the panel
  3. once enabled/wallet created, update to 0.68.136 Chromium: 77.0.3865.75
  4. click on the reward panel and you'll see "Join Rewards" even though the wallet is created

Actual result:

rewardsIssue

Screen Shot 2019-09-13 at 4 43 10 PM

Expected result:

Once you've joined rewards and created a new wallet, the panel shouldn't be displaying "Join Rewards" after a restart.

Reproduces how often:

100% reproducible when going through the above STR.

Brave version (brave://version info)

Brave 0.68.132 Chromium: 76.0.3809.132 (Official Build) (64-bit)
Revision fd1acc410994a7a68ac25bc77513d443f3130860-refs/branch-heads/3809@{#1035}
OS Mac OS X

Version/Channel Information:

  • Can you reproduce this issue with the current release? No, not reproducible using 0.68.132 Chromium: 76.0.3809.132 (Official Build
  • Can you reproduce this issue with the beta channel? No, not reproducible using 0.70.93 Chromium: 76.0.3809.132 (Official Build) beta
  • Can you reproduce this issue with the dev channel? Yes, reproducible using 0.71.73 Chromium: 77.0.3865.75 (Official Build) dev
  • Can you reproduce this issue with the nightly channel? Yes, reproducible using 0.72.23 Chromium: 77.0.3865.75 (Official Build) nightly

Other Additional Information:

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

Miscellaneous Information:

CCing @rebron @GeetaSarvadnya @NejcZdovc @bsclifton

@kjozwiak kjozwiak added bug release/blocking feature/rewards priority/P1 A very extremely bad problem. We might push a hotfix for it. QA/Yes QA/Test-Plan-Specified labels Sep 13, 2019
@kjozwiak kjozwiak added this to the 0.68.x - Release Hotfix 2 milestone Sep 13, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Sep 14, 2019
@bsclifton
Copy link
Member

Fixed in 0.68.x with brave/brave-core#3432

That patch needs to be PRed against master / 0.71.x / 0.70.x / 0.69.x

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 16, 2019

Verification passed on

Brave 0.68.138 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)
  • Verified the STR from the description

Verification passed on

Brave 0.68.138 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Ubuntu 18.04 LTS
  • Verified the STR from the description

Verification PASSED on macOS 10.14.6 x64 using the following build:

Brave 0.68.138 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS macOS Version 10.14.6 (Build 18G95)

@rebron
Copy link
Collaborator

rebron commented Sep 16, 2019

Fix looks good on 0.68.138 Chromium: 77.0.3865.75 (Official Build) (64-bit), macOS with old and new profile.

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

Successfully merging a pull request may close this issue.

6 participants