Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

'Claim my rewards' button on modal doesn't work the first time #2280

Closed
LaurenWags opened this issue Jan 29, 2020 · 6 comments
Closed

'Claim my rewards' button on modal doesn't work the first time #2280

LaurenWags opened this issue Jan 29, 2020 · 6 comments

Comments

@LaurenWags
Copy link
Member

Description:

Found while testing #2255

The first time you get the Claim my rewards modal on a NTP and you tap the button, your rewards aren't claimed. You have to either a) claim from the panel or b) trigger this modal again. On triggering it again the second claim will be successful.

Steps to Reproduce

  1. install 1.14.3 (20.1.29.20) via the Dev TF channel
  2. skip the Onboarding process and go into Settings --> View Rewards Debug Menu
  3. select Staging and restart the browser (should close automatically)
  4. once the browser is relaunched, enable rewards via the panel (Join Rewards)
  5. go into Settings --> View Rewards Debug Menu
  6. restore a wallet that has an ad grant available (use the shared GDoc) and restart the browser
  7. open the panel and then a NTP and you'll get the Brave Reward modal that displays your ad grant
  8. Tap on Claim my rewards button.

Actual result:
Modal closes, rewards are not claimed.
You must either:
a. Tap on the panel and claim from there
OR
b. Close the app, remove from memory. Relaunch, open the panel, and then open a NTP. This will trigger the modal again and claiming from this second modal will work.

Expected result:
Claim should work from modal the first time.

Reproduces how often: [Easily reproduced, Intermittent Issue]
easily

Brave Version:
1.14.3 (20.1.29.20)

Device details:
iPad 5th Gen iOS12 and iPad mini iOS13

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

cc @brave/legacy_qa @jhreis @iccub

@kjozwiak
Copy link
Member

@kylehickinson pointed out that this might be related/the same issue as #1998.

@iccub
Copy link
Contributor

iccub commented Jan 30, 2020

Also wanted to note that with current implementation, the 'Claim Rewards' notification on NTP is attempted on every app launch.
So if it fails on first claim, the user still gets the prompt next time, until there's grant no available

@iccub
Copy link
Contributor

iccub commented Jan 30, 2020

Bumping to next release as we are still not sure what is source of the problem

@iccub iccub modified the milestones: 1.14.3, 1.15 Jan 30, 2020
@kylehickinson
Copy link
Collaborator

Duplicate of #1998

@kylehickinson kylehickinson marked this as a duplicate of #1998 Jan 30, 2020
@iccub
Copy link
Contributor

iccub commented Feb 7, 2020

Just tested on prod and it works correctly, closing for now. Reopen if this happens on prod to you

@srirambv
Copy link
Contributor

Verification passed on iPhone XR with iOS 13.4 running 1.15(20.02.18.17)

  • Verified able to claim UGP grant in the first try

Verification passed on iPhone 7+ with iOS 13.3.1 running 1.15(20.02.18.17)

  • Verified able to claim UGP grant in the first try

Verification passed on iPhone 6 with iOS 12.4.5 running 1.15(20.02.18.17)

  • Verified able to claim UGP grant in the first try

Verification passed on iPad Pro with iOS 12.4.5 running 1.15(20.02.18.17)

  • Verified able to claim UGP grant in the first try

Verification passed on iPad Pro with iOS 13.3.1 running 1.15(20.02.18.17)

  • Verified able to claim UGP grant in the first try

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

No branches or pull requests

5 participants