-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
grandfathered unverified state does not display processing or complete payout banners #28001
Labels
feature/rewards
OS/Android
Fixes related to Android browser functionality
OS/Desktop
priority/P1
A very extremely bad problem. We might push a hotfix for it.
QA Pass - Android ARM
QA Pass-macOS
QA/Yes
regression
release-notes/exclude
Milestone
Comments
LaurenWags
added
feature/rewards
QA/Yes
OS/Android
Fixes related to Android browser functionality
OS/Desktop
labels
Jan 24, 2023
Closed
@brave/qa-team when retesting this, we should also re-verify test case 15 from QA doc for "Rewards payout banner status (non-connected state)" to ensure that is still working as expected since that is also an unverified state. |
Miyayes
added
the
priority/P1
A very extremely bad problem. We might push a hotfix for it.
label
Jan 24, 2023
25 tasks
7 tasks
The above requires |
LaurenWags
added
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
QA Pass-macOS
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Jan 30, 2023
Verification PASSED on
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/rewards
OS/Android
Fixes related to Android browser functionality
OS/Desktop
priority/P1
A very extremely bad problem. We might push a hotfix for it.
QA Pass - Android ARM
QA Pass-macOS
QA/Yes
regression
release-notes/exclude
Description
The grandfathered unverified state does not display "processing" or "complete" payout banners. It does display the "pending" banner.
See test case 16 from QA doc (see @Miyayes for access to doc)
Steps to Reproduce
16.1 Create rewards profile on older version of Brave and view some ads. Then update to 1.48.x so that you’re in the grandfathered-unverified state.
16.2 View couple of ads and confirm ads are cashed in with the server (verify
Successfully redeemed unblinded payment tokens
msg in logs)16.3 Look for the last message similar to "You have X unblinded payment tokens which will be redeemed in (some hours, minutes, seconds) at (some time)" message, once this time has come, you should then see a response in the logs for "OnRedeemUnblindedPaymentTokens". If this is successful, you can proceed to next steps.
16.4 Close browser. Advance clock to the next month to the 2nd (or between 1st and 7th), and then relaunch the browser, to see if you see any “Payout will arrive in x days” banner”. This banner displays as expected:
16.5 Advance your clock to the 8th or later and then relaunch the browser.
16.6 Ask someone on server team, or use Charles Proxy to set the payout status at the staging /parameters endpoint to “processing” for unverified users.
16.7 Make sure that the browser has picked up this new /parameters value.
Logs:
---> 16.8 The “processing” banner does not appear in the Rewards panel, NTP widget, and brave://rewards page as it should:
16.9 Ask someone on server team, or use Charles Proxy to set the payout status at the staging /parameters endpoint to “complete” for unverified users.
16.10 Make sure that the browser has picked up this new /parameters value.
Logs:
---> 16.11 The “complete” banner does not appear in the Rewards panel, NTP widget, or brave://rewards page as expected
16.12 Bring your clock back to the current month to 9th or later, and then relaunch the browser, and make sure you don’t see the banners (because it’s the wrong month)
Actual result:
see above, screenshots are in line w/ steps
Expected result:
processing and complete banners should show in steps 8 and 11 respectively.
Here's an example from 1.47.x of the "complete" banner:
Reproduces how often:
easily
Desktop Brave version:
1.48.x
Android Device details:
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @Miyayes @brave/qa-team
The text was updated successfully, but these errors were encountered: