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

BR panel keeps showing 1 notification message even when rewards is not turned on #2957

Closed
srirambv opened this issue Jan 15, 2019 · 6 comments · Fixed by brave/brave-core#1486

Comments

@srirambv
Copy link
Contributor

Description

BR panel keeps showing 1 notification message even when rewards is not turned on

Steps to Reproduce

  1. Clean profile on Linux
  2. BR panel shows 1 notification
  3. Click on BR panel, notification goes away
  4. Click on BR panel again notification comes back

Actual result:

BR panel keeps showing 1 notification message even when rewards is not turned on

Expected result:

Should not show 1 notification message when rewards is turned off

Reproduces how often:

Easy on Linux

Brave version (brave://version info)

Brave 0.60.13 Chromium: 72.0.3626.53 (Official Build) dev(64-bit)
Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620}
OS Linux

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    Yes on Dev build

Website problems only:

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

Additional Information

@brave/legacy_qa to verify on other platforms @NejcZdovc

@btlechowski
Copy link

Not reproducible on Windows 7 x64. The notification goes away when brave panel is opened for the first time and never comes back. Tried reloading the page, restarting the browser.

Brave 0.60.13 Chromium: 72.0.3626.53 (Official Build) dev (64-bit)
Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620}
OS Windows

@NejcZdovc NejcZdovc added the priority/P4 Planned work. We expect to get to it "soon". label Jan 19, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Jan 29, 2019
@NejcZdovc
Copy link
Contributor

reproduced on macOS and it's a regression from this PR brave/brave-core#1409, so we need this fix in 0.60

cc @kjozwiak @rebron

@kjozwiak
Copy link
Member

kjozwiak commented Feb 6, 2019

reproduced on macOS and it's a regression from this PR brave/brave-core#1409, so we need this fix in 0.60

cc @kjozwiak @rebron

@NejcZdovc are we still considering uplifting this into 0.60.x?

@NejcZdovc
Copy link
Contributor

@kjozwiak I would uplift it yes. Let me create PR for it

@NejcZdovc
Copy link
Contributor

@kjozwiak created PR brave/brave-core#1588

@NejcZdovc NejcZdovc modified the milestones: 0.61.x - Dev, 0.60.x - Beta Feb 7, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Feb 8, 2019

Verified passed with

Brave 0.60.23 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Mac OS X

Verification passed on

Brave 0.60.23 Chromium: 72.0.3626.96 (Official Build) beta (64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Windows 10
  • Verified 1 notification is gone from BAT logo when rewards is enabled through welcome screen
  • Verified 1 notification is gone from BAT logo when rewards is enabled through Hamburger menu,

Verification passed on

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Linux

Used test plan from brave/brave-core#1486

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