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

Unable to dismiss the first rewards notification #6691

Closed
btlechowski opened this issue Oct 31, 2019 · 3 comments · Fixed by brave/brave-core#3898
Closed

Unable to dismiss the first rewards notification #6691

btlechowski opened this issue Oct 31, 2019 · 3 comments · Fixed by brave/brave-core#3898

Comments

@btlechowski
Copy link

btlechowski commented Oct 31, 2019

The first rewards notification reappears after browser restart

Steps to Reproduce

  1. Clean install
  2. Click BAT icon in URL bar -> first rewards notification is dismissed (expected)
  3. Restart Brave

Actual result:

first rewards notification is shown
image

Expected result:

first rewards notification is dismissed
image

Broader test plan (from the PR via @petemill)

  1. Clean user data directory
  2. Launch Brave
  3. Create a second profile via App Menu -> Create a New Profile
  4. Expected: Both profile windows have the rewards icon with the badge text "1"
  5. Click the rewards button in the second profile window
  6. Expected: The second profile window's rewards icon does not have a badge
  7. Expected: The first profile window's rewards icon does have a badge
  8. Quit the browser
  9. Start the browser again
  10. Expected: The rewards icon badge state from each profile window is the same as 5) and 6)

Reproduces how often:

Always

Brave version (brave://version info)

Reproducible on

Brave 0.71.106 Chromium: 78.0.3904.70 (Official Build) beta (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS
Brave 0.72.112 Chromium: 78.0.3904.70 (Official Build) dev (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS
Brave 0.73.45 Chromium: 78.0.3904.70 (Official Build) nightly (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS

Not reproducible on

Brave 0.70.121 Chromium: 78.0.3904.70 (Official Build) (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS

cc @brave/legacy_qa @NejcZdovc @petemill

@btlechowski
Copy link
Author

@rebron @bsclifton While this is a visual only issue, it is a regression. Could you prioritize it?

@petemill petemill self-assigned this Nov 5, 2019
@petemill petemill moved this from New Issues to In Progress in Rewards Nov 5, 2019
@rebron rebron added 1.0-suggestion priority/P2 A bad problem. We might uplift this to the next planned release. and removed 1.0-suggestion labels Nov 5, 2019
@bsclifton bsclifton added this to the 0.74.x - Nightly milestone Nov 6, 2019
@bsclifton
Copy link
Member

marked as release-notes/exclude since it's not reproducible on 0.70.x

@LaurenWags
Copy link
Member

LaurenWags commented Nov 8, 2019

Verified passed with

Brave 0.71.113 Chromium: 78.0.3904.87 (Official Build) (64-bit)
Revision 20c21f4010010f32462ea8e1d6af30cef66d48c8-refs/branch-heads/3904@{#840}
OS macOS Version 10.13.6 (Build 17G5019)

Verification passed on

Brave 0.71.114 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verification passed on

Brave 0.71.114 Chromium: 78.0.3904.97 (Official Build) (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Ubuntu 18.04 LTS

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