-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
"Complete" payout status is not showing green "Arrived" message banner #24980
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
release/blocking
release-notes/exclude
Milestone
Comments
LaurenWags
added
release/blocking
feature/rewards
QA/Yes
release-notes/exclude
OS/Android
Fixes related to Android browser functionality
OS/Desktop
labels
Aug 25, 2022
Miyayes
added
the
priority/P1
A very extremely bad problem. We might push a hotfix for it.
label
Aug 25, 2022
25 tasks
This was referenced Aug 26, 2022
Added |
Can be tested with 1.43.84 and above. |
LaurenWags
added
the
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
label
Aug 30, 2022
LaurenWags
added
QA Pass-macOS
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Aug 30, 2022
Uni-verse
added
the
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
label
Aug 30, 2022
Verification completed on
Testing based on #24980 (comment) 1. For the below cases, confirmed when "complete" is returned from payoutStatus API, and it is on or after the payout date (ex. 7th in my case), and ad tokens were cashed in for the previous month, the "Arrived" message shows as expected:Unverified/Anon - PASSEDWhen using an unverified/anon setup for rewards (no custodial provider):
Logs:
Gemini - PASSED
Logs:
Uphold - PASSED
Logs:
bitFlyer - PASSED
Logs:
|
Uni-verse
added
QA Pass - Android ARM
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Aug 31, 2022
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
release/blocking
release-notes/exclude
Description
Found while testing #23429
When
parameters
endpoint returnscomplete
(and the date is on or after the payment date), there is no green "Arrived" banner for ad grant. Green "Arrived" banner does not display until you successfully complete the captcha. This seems to indicate that the API isn't being utilized.Steps to Reproduce
Pre-req: server should be set to return "complete" for payout status
Repeat above for unverified rewards as well as verified user w/ a custodial wallet provider (Uphold, Gemini, bF)
Actual result:
No green "Arrived" banner on ads panel of brave://rewards, rewards panel, NTP widget
Expected result:
Green "Arrived" banner to show since the API is returning "complete"
Note - possibly the green "Arrived" banner should be removed from view if you complete a captcha? Mentioned by @Miyayes. Need to confirm - makes sense for it to do so in unverified case, but what happens if you have a custodian? does the banner just always display until "off" is returned from API?
Reproduces how often:
easily
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @Miyayes @emerick @GeetaSarvadnya
The text was updated successfully, but these errors were encountered: