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

[Android] Promotion notification is not cleared when error happens #9657

Closed
NejcZdovc opened this issue May 6, 2020 · 4 comments · Fixed by brave/brave-core#5520
Closed

Comments

@NejcZdovc
Copy link
Contributor

Description

User clicks on ad claim button in the panel. Error happens in claim flow (for this particular case in unblinding code) but user don't receive any feedback, just spinner.

Steps to reproduce

Actual result

Expected result

If there is any error in claim flow we need to pass this back to the user and clear notification. So when we fetch promotions next time we get an actual result from the server and not something that is cached

Issue reproduces how often

Issue happens on

  • Current Play Store version?
  • Beta build?

Device details

  • Install type (ARM, x86):
  • Device (Phone, Tablet, Phablet):
  • Android version:

Brave version

Website problems only

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

Additional information

@NejcZdovc NejcZdovc added needs-investigation A bug not 100% confirmed/fixed feature/rewards OS/Android Fixes related to Android browser functionality labels May 6, 2020
@gdregalo
Copy link

gdregalo commented May 7, 2020

@NejcZdovc
If multiple promotions are available, dismissing a promotion UI will open another promotion ready to be claimed. Is it safe to start the second claim while the first one is still being processed?

@NejcZdovc
Copy link
Contributor Author

yes it's safe

@btlechowski
Copy link

btlechowski commented Jun 17, 2020

Verification passed on BraveModernarm v1.10.92 on Samsung Galaxy Tab S5e (Android 9.0)

Verified test plan from brave/brave-core#5520
Error message after tapping Claim Grant
Screenshot_20200617-053418_Brave

Error message after showing Rewards panel
Screenshot_20200617-053329_Brave

Verification passed on Samsung Galaxy J3 version 5.1.1 running BraveModernarm 1.10.92.apk

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.

4 participants