-
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
Implement new Rewards NTP Widget v2 design #17483
Comments
Proposed test cases for this issue: Figma doc for reference: https://www.figma.com/file/oWF3SKW1KSQxFpSKPskpoh/Brave-Rewards-widget?node-id=42%3A1943. Scenario 1 - br v2 (normal)A. Clean profile
B. Upgrade profile
Scenario 2 - br v2 (pending)Note before beginning, the "pending" message has slightly different text than figma. Confirmed with Chris/Kevin this is expected. Wording in this area was generally changed with #17943. A. Clean profile
B. Upgrade profile
Scenario 3 - br v2 (arrived kyc'd) - N/AConfirmed with Kevin that this scenario is not applicable at this time. Scenario 4 - br v2 (arrived grant)A. Clean profile 1 - closing ad grant message without claim
B. Clean profile 2 - claiming ad grant from widget
C. Clean profile 3 - multiple ad grants
D. Upgrade profile - ad grant
E. Spot check UGP grants
Scenario 5 - br v2 (earn and give)Note: "proceeding" text instead of "click" is expected, confirmed with Chris/Kevin. A. Clean profile
B. Upgrade profile
Scenario 6 - br v2 (enable ads)A. Clean profile
B. Upgrade profile
Scenario 7 - Earning tooltip ("i" icon)Note - text differs slightly from figma, this was changed with #17943 to use the "processing" language. A. Clean profile
Scenario 8 - NTP SI TooltipNote - as with above scenarios, some wording differs slightly from figma. This is expected per discussions. A. Clean profile 1
B. Clean profile 2
C. Clean profile 3
D. Clean profile 4
E. Clean profile 5
F. Upgrade profile 1
G. Upgrade profile 2
Scenario 9 - TranslationsSpot check translations on the widget in several languages. |
Verified using
Verified various UI states from https://www.figma.com/file/oWF3SKW1KSQxFpSKPskpoh/Brave-Rewards-widget?node-id=42%3A1943. Logged #19691. Scenario 1 - br v2 (normal) - PASSEDA. Clean profile
Confirmed Balance, Earning and Giving amounts were as expected per the panel and brave://rewards page.
B. Upgrade profile
Confirmed Balance, Earning and Giving amounts were as expected per the previous version.
Scenario 2 - br v2 (pending) - PASSEDA. Clean profile
Confirmed "pending" messaging was displayed on widget (note, different text than figma. Confirmed with Chris/Kevin this is expected. Wording in this area was generally changed with #17943).
B. Upgrade profile
Confirmed "pending" messaging was displayed on widget (note, different text than figma as per above).
Scenario 3 - br v2 (arrived kyc'd) - N/AConfirmed with Kevin that this scenario is not applicable at this time. Scenario 5 - br v2 (earn and give) - PASSEDA. Clean profile
Confirmed "Terms of Service" link directs to https://basicattentiontoken.org/user-terms-of-service/.
B. Upgrade profile
Confirmed "Terms of Service" link directs to https://basicattentiontoken.org/user-terms-of-service/.
Scenario 6 - br v2 (enable ads) - PASSEDA. Clean profile
Confirmed UI was as per figma.
B. Upgrade profile
Confirmed UI was as per figma.
Scenario 7 - Earning tooltip ("i" icon) - PASSEDA. Clean profile
Note - text differs slightly from figma, this was changed with #17943 to use the "processing" language. Confirmed UI was as per figma.
Scenario 8 - NTP SI Tooltip - PASSEDA. Clean profile 1
Confirmed "Terms of Service" link directs to https://basicattentiontoken.org/user-terms-of-service/. B. Clean profile 2
Confirmed NTP SI tooltip was not shown on subsequent NTP SI pages once it had been visible for ~5s. C. Clean profile 3
Confirmed UI was as per figma. D. Clean profile 4
Confirmed tooltip was displayed in step where expected. E. Clean profile 5
Confirmed tooltips were shown independent of the widget. F. Upgrade profile 1
Confirmed no tooltip on upgraded profile when the previous message had been displayed on earlier version. G. Upgrade profile 2
Confirmed no tooltip on upgraded profile when the previous message had been displayed on earlier version. Scenario 4 - br v2 (arrived grant) - PASSEDA. Clean profile 1 - closing ad grant message without claim
Confirmed widget showed appropriate message per figma.
B. Clean profile 2 - claiming ad grant from widget
Confirmed widget showed appropriate message per figma.
C. Clean profile 3 - multiple ad grants
Confirmed multiple ad grant messages can be handled on the widget.
D. Upgrade profile - ad grant
Confirmed widget showed appropriate message per figma.
E. Spot check UGP grants
Confirmed the above work as expected. Scenario 9 - Translations - PASSEDSpot check translations on the widget in several languages.
Verification passed on
Scenario 1 - br v2 (normal) - PASSEDA. Clean profile
Confirmed Balance, Earning and Giving amounts were as expected per the panel and brave://rewards page.
B. Upgrade profile
Confirmed Balance, Earning and Giving amounts were as expected per the previous version.
Scenario 2 - br v2 (pending) - PASSEDA. Clean profile
Confirmed "pending" messaging was displayed on widget (note, different text than figma. Confirmed with Chris/Kevin this is expected. Wording in this area was generally changed with #17943).
Scenario 4: br v2 (arrived grant) - PASSEDB. Clean profile 2 - claiming ad grant from widget
Confirmed widget showed appropriate message per figma.
(Claimed two grants(5 BAT and 3 BAT) as @Miyayes sent two ad grants. hence balance 8 BAT is shown in the NTP widget Scenario 5 - br v2 (earn and give) - PASSEDA. Clean profile
Confirmed "Terms of Service" link directs to https://basicattentiontoken.org/user-terms-of-service/.
Scenario 6 - br v2 (enable ads) - PASSEDA. Clean profile
Confirmed UI was as per figma.
Scenario 7 - Earning tooltip ("i" icon) - PASSEDA. Clean profile
Note - text differs slightly from figma, this was changed with #17943 to use the "processing" language. Confirmed UI was as per figma.
Scenario 8 - NTP SI Tooltip - PASSEDB. Clean profile 2
Confirmed NTP SI tooltip was not shown on subsequent NTP SI pages once it had been visible for ~5s. C. Clean profile 3
Confirmed UI was as per figma. E. Clean profile 5
Confirmed tooltips were shown independent of the widget. F. Upgrade profile 1
Confirmed no tooltip on upgraded profile when the previous message had been displayed on earlier version. G. Upgrade profile 2
Confirmed no tooltip on upgraded profile when the previous message had been displayed on earlier version. Verified
As discussed in Slack, for Linux, I spot-checked against a few various states in Figma: https://www.figma.com/file/oWF3SKW1KSQxFpSKPskpoh/Brave-Rewards-widget?node-id=42%3A1943
|
Description
Implement the new Rewards NTP Widget v2 design.
Message @Miyayes or @karenkliu for spec.
The text was updated successfully, but these errors were encountered: