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.

Sign up for GitHub

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

Jump to bottom

Add onboarding flow for new users. #297

Merged
merged 49 commits into from
Aug 26, 2024
Merged

Add onboarding flow for new users. #297

merged 49 commits into from
Aug 26, 2024

Conversation

StevenDufresne
Copy link
Contributor

@StevenDufresne StevenDufresne commented Aug 14, 2024 •
edited
Loading

This builds on #286 but adds code to make the wizard inline.

Screenshots

profiles wordpress org_dufresnesteven_profile_security_first-time=1 screen=home (1)

profiles wordpress org_dufresnesteven_profile_security_first-time=1 screen=home (2)

profiles wordpress org_dufresnesteven_profile_security_first-time=1 screen=home (3)

profiles wordpress org_me_profile_security (1)

profiles wordpress org_me_profile_security (2)

StevenDufresne and others added 30 commits August 14, 2024 11:29
@StevenDufresne
Try a setup wizard.
6da018f
@StevenDufresne
More changes.
64d38ef
@StevenDufresne
More changes
a3fc16e
@StevenDufresne
Fix the linter.
53131c5
@StevenDufresne
Clean up
7b4c749
@StevenDufresne
Update account-status to be 'home'.
7aaecf8
@StevenDufresne
Add back empty line.
98335e8
@StevenDufresne
Update copy.
e347ab3
@StevenDufresne
Add redirect logic.
3673f39
@StevenDufresne
Fix spacing.
16ab35f
@StevenDufresne
Improve css.
a4152b5
@StevenDufresne
Font show if they have a primary.
0db3f19
@StevenDufresne
We can't check via the component because it reloads.
7d8b9e1
@StevenDufresne
Remove eslint disable line.
70aa9c4
@StevenDufresne
Remove empty line.
213d2d4
@StevenDufresne
Clean up formatting.
8c1e2ea
@StevenDufresne
Revert "Clean up formatting."
e88b9c8 
This reverts commit 0b63573.
@StevenDufresne
clean up formatting.
a13344b
@StevenDufresne
Lowercase setup radial.
a494064
@StevenDufresne
Trap the keyboard focus within the window.
919094f
@StevenDufresne
Update the padding-top for the wizard.
21213a9
@StevenDufresne
Update the component name.
6d9b0b6
@StevenDufresne @adamwoodnz
Update settings/src/components/first-time/first-time.js
f6e3287 
Co-authored-by: Adam Wood <1017872+adamwoodnz@users.noreply.github.com>
@StevenDufresne
Prefer keys as the first option.
6a5aaaa
@StevenDufresne
Move progress indicator about window.
e8ab11e
@StevenDufresne
Default to 2fa view.
f4cd78b
@StevenDufresne
move the progress component.
ea309fd
@StevenDufresne
Update selected radial to be webauthn.
5268407
@StevenDufresne
Fix react error with map key.
4354dbe
@StevenDufresne
Update radio list styles.
8becb71
tellyworth
tellyworth reviewed Aug 14, 2024
View reviewed changes
Copy link

@tellyworth tellyworth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some nitpicky copy suggestions

settings/src/components/first-time/congratulations.js Outdated Show resolved Hide resolved
settings/src/components/first-time/congratulations.js Outdated Show resolved Hide resolved
settings/src/components/first-time/home.js Show resolved Hide resolved
settings/src/components/first-time/first-time.js Show resolved Hide resolved
settings/src/components/first-time/first-time.js Outdated Show resolved Hide resolved
settings/src/components/first-time/congratulations.js Outdated Show resolved Hide resolved
@StevenDufresne StevenDufresne mentioned this pull request Aug 16, 2024
Merged
@StevenDufresne StevenDufresne changed the title Experiment: Alternative to #286 wizard. Add onboarding flow for new users. Aug 16, 2024
@StevenDufresne StevenDufresne mentioned this pull request Aug 22, 2024
Closed
@StevenDufresne StevenDufresne marked this pull request as ready for review August 22, 2024 06:22
@StevenDufresne StevenDufresne merged commit 8c63862 into trunk Aug 26, 2024
2 checks passed
@StevenDufresne StevenDufresne deleted the try/inline branch August 26, 2024 03:39
@StevenDufresne StevenDufresne mentioned this pull request Aug 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tellyworth tellyworth tellyworth left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@StevenDufresne @tellyworth