-
Notifications
You must be signed in to change notification settings - Fork 900
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
Improve rewards onboarding experience #7135
Conversation
- Show rewards onboarding UX in various UI parts independently - Do not show rewards onboarding to existing opted-in rewards users
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! I left a question for curiosity sake, but no changes requested.
components/brave_rewards/resources/shared/lib/onboarding_completed_store.ts
Show resolved
Hide resolved
Verified passed with
Verified test plan from #7135 (comment) Verified steps from
ScreenshotsPanel onboarding: Panel after dismissing onboarding: Banner onboarding: Banner after dismissing onboarding: brave://rewards onboarding: brave://rewards after dismissing onboarding: Upgrade Case 1 - Rewards never enabled on 1.17.x
Upgrade Case 2 - Rewards enabled (AC + Ads enabled) on 1.17.x
Upgrade Case 3 - Rewards enabled, then disabled on 1.17.x
Upgrade Case 4 - Rewards enabled (Ads on, AC off) on 1.17.x
Upgrade Case 5 - Rewards enabled (Ads off, AC on) on 1.17.x (
Upgrade Case 6 - Rewards enabled (AC + Ads off) on 1.17.x
|
This PR introduces the following changes:
Resolves brave/brave-browser#12675
Resolves brave/brave-browser#12612
Submitter Checklist:
npm run lint
,npm run gn_check
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).QA/Yes
orQA/No
) to the associated issuerelease-notes/include
orrelease-notes/exclude
) to the associated issueTest Plan:
Showing onboarding at different entry points independently
Note: the following steps should demonstrate the same behavior, regardless of the order of the entry points visited (the rewards page, the rewards panel, or the tip dialog).
Click the "X" to close the tip dialog.Click the "Maybe Later" link to dismiss onboarding.Onboarding is not displayed for upgraded users with Rewards enabled
Reviewer Checklist:
After-merge Checklist:
changes has landed on.