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

brave notifying users there's a grant available even though there's none available #1793

Closed
kjozwiak opened this issue Oct 23, 2018 · 3 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Oct 23, 2018

Description

When we were testing #1350 tonight, several internal employees including myself started running into CAPTCHA issues. We couldn't complete the CAPTCHA and figured the fix wasn't working. However, @evq mentioned that the server ran out of grants so there weren't any to accept.

We shouldn't be notifying issues when there's no grants available.

Steps to Reproduce

We'll need to create a scenario on staging where there's no more grants available. Launching brave with a new profile in this scenario shouldn't notify the user that there's grants available.

  • ensure that the above scenario is setup (most likely on staging)
  • install brave and enable rewards (you shouldn't be seeing any notifications)
  • add more grants to the backend
  • restart brave with a new profile and ensure that the notifications are now being displayed

Actual result:

Being notified that there's a grant available even though the server has ran out of grants.

Expected result:

Users shouldn't be notified about grants if there's none left on the server.

Reproduces how often:

Assuming it's 100% as @mandar-brave @yrliou @jenn-rhim and myself reproduced the issue over and over again.

Brave version (brave://version info)

Brave 0.55.20 Chromium: 70.0.3538.67 (Official Build) (64-bit)
Revision 9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS Mac OS X

Additional Information

CCing @evq

@evq
Copy link
Member

evq commented Oct 25, 2018

fixed in brave-intl/bat-ledger#422

@LaurenWags
Copy link
Member

LaurenWags commented Nov 7, 2018

Verified passed with

Brave 0.56.9 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Mac OS X
  • Verified STR from description

Verification passed on

Brave 0.56.9 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Windows 7 x64

Went through verification using the following build under Ubuntu 18.04 x64 - PASSED

Brave 0.56.9 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Linux

@kjozwiak
Copy link
Member Author

kjozwiak commented Nov 7, 2018

QA worked with @maikelmclauflin to verify that this has been fixed. The staging server was put into a state where the promotion was live, but grants ran out. At this point, QA launched Brave using the staging server and ensured that we didn't receive any notifications regarding grants. Once we confirmed the above, @maikelmclauflin reloaded staging with more grants and QA confirmed that the notifications being displayed. We also ensured that the grants were redeemable.

Test Case that QA ran through on macOS, Win and Linux:

  • @maikelmclauflin put the staging server into a state where the promotion was live but grants ran out
  • QA installed Brave on all three platforms and ensured that we weren't receiving notifications
  • @maikelmclauflin re-added grants on the staging server
  • QA ensured that that refreshing brave://rewards displayed the notification that there's grants available
  • ensured that the grants were also redeemable

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