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

One-time tip data reset to Zero after upgrade/browser restart #5987

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

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 11, 2019

Description

One-time tip data reset to Zero after upgrade/browser restart, refresh brave://rewards page resolves the issue
This issue is only happening with chromium 77. The issue is Reproducible in both Nightly and Release

0.68.134 Chromium: 77.0.3865.75 -Release - Reproduced
0.72.11 Chromium: 77.0.3865.65-Nightly - Reproduced
0.68.132 Chromium: 76.0.3809.132- Released - Not Reproducible
0.71.72 Chromium: 76.0.3809.132- Dev - Not Reproducible
0.70.93 Chromium: 76.0.3809.132 -Beta - Not Reproducible

Steps to Reproduce

STR1:

 1. Clean profile 0.68.134
 2. Enable rewards and restore a key which has the balance
 3. Send a one time tip to verified pubs
 4. Click on `Exit` in the hamburger menu
 5. Restart the browser
 6. Tip data reset to zero in both tips panel as well as in rewards summary section
 7. Refresh brave://rewards page
 8. Tip data is shown as expected   

STR2:

 1. Clean profile 0.68.132
 2. Enable rewards and restore a key which has the balance
 3. Send a one-time tip to a verified publisher
 4. Upgrade to 0.68.134
 5. One-time tip data reset to Zero after upgrade
 6. Refresh brave://rewards page
 7. Tip data is shown as expected

Actual result:

Tip panel:

image

Rewards summary:

image

Expected result:

One-time tip data shouldn't reset to Zero after upgrade/browser restart

Reproduces how often:

Always

Brave version (brave://version info)

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

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the dev channel? No
  • 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:

@brave/legacy_qa @NejcZdovc

@LaurenWags
Copy link
Member

LaurenWags commented Sep 11, 2019

If I don't have the brave://rewards page open on upgrade this doesn't reproduce for me. If my brave://rewards page is open, the issue does reproduce. However, refreshing the brave://rewards page doesn't fix it for me, I need to close and reopen that page.

The steps I used were:

  1. Clean profile 0.68.134
  2. Enable rewards and restore a key which has the balance
  3. Send a one time tip to verified pubs
  4. Keep Rewards page open and in focus. Close Brave from top menu bar or cmd+Q (macOS as there is no Exit in hamburger menu on macOS).
  5. Restart the browser
  6. Tip data reset to zero in rewards summary section
  7. Close and reopen brave://rewards page
  8. Tip data is shown as expected
Brave 0.68.134 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS macOS Version 10.13.6 (Build 17G5019)

@LaurenWags
Copy link
Member

LaurenWags commented Sep 11, 2019

Also, I encountered #5336 a lot while trying to reproduce this issue. Unsure if maybe the white box is easier to get on this version, or if it was my steps, but here's what I was doing:

Scenario 1

  1. Clean profile 0.68.134
  2. Enable rewards and restore a key which has the balance
  3. Send a one time tip to verified pubs
  4. Keep Rewards page open and in focus. Close Brave from top menu bar or cmd+Q (macOS as there is no Exit in hamburger menu on macOS).
  5. Restart the browser
  6. Tip data reset to zero in rewards summary section
  7. Click on panel.
    --> Error message shown over wallet balance on brave://rewards and white box for panel. Error message goes away, but white box on panel is permanent and cannot be recovered from.

Scenario 2:

  1. Clean profile 0.68.134
  2. Enable rewards from panel. Don't visit brave://rewards page.
  3. Close Brave from top menu bar or cmd+Q (macOS as there is no Exit in hamburger menu on macOS).
  4. Restart the browser
  5. Click on panel.
    --> White box for panel. White box on panel is permanent and cannot be recovered from.

@NejcZdovc
Copy link
Contributor

@LaurenWags 5336 was fixed in 69, so maybe worth talking about pulling it into 68 + 77

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Sep 11, 2019

@LaurenWags I have encountered the issue #5336 twice today, STR is mentioned below.

  1. Clean profile 0.68.132 staging flags
  2. Complete the auto contribution
  3. Upgrade to 0.68.134
  4. Restart the browser and launch using staging flags
  5. Click on BR panel showing the white popup
  6. Complete the auto contribution in .134
  7. Click on BR panel unable to see a-c summary in BR panel due to white popup

@kjozwiak kjozwiak removed this from the 0.68.x - Release Hotfix 2 milestone Sep 11, 2019
@kjozwiak
Copy link
Member

Removing from the 0.68.x release but we should definitely fix this sooner than later. Users might get confused when the state doesn't update right away whenever Rewards is being used.

@NejcZdovc NejcZdovc added the priority/P4 Planned work. We expect to get to it "soon". label Sep 12, 2019
@kjozwiak
Copy link
Member

@NejcZdovc moving this back into 0.68.x. With 0.68.136 Chromium: 77.0.3865.75, restarting will clear all the data via Activity Summary as mentioned above but this time around, refreshing brave://rewards won't restore the lost data. Example of the issue occurring:

rewardsIssue

@kjozwiak kjozwiak added priority/P1 A very extremely bad problem. We might push a hotfix for it. release/blocking and removed priority/P4 Planned work. We expect to get to it "soon". labels Sep 13, 2019
@kjozwiak kjozwiak added this to the 0.68.x - Release Hotfix 2 milestone Sep 13, 2019
@kjozwiak
Copy link
Member

@NejcZdovc as per the above, once the data disappears via Activity Summary, you won't be able to get it back with a refresh of brave://rewards.

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

@NejcZdovc NejcZdovc self-assigned this Sep 15, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Sep 15, 2019
@GeetaSarvadnya
Copy link
Author

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)

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

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)

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