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.

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 the create account onboarding flow #140

Merged
merged 25 commits into from
Jan 13, 2021
Merged

Improve the create account onboarding flow #140

merged 25 commits into from
Jan 13, 2021

Conversation

MaxwellBo
Copy link
Owner

@MaxwellBo MaxwellBo commented Jan 9, 2021

After

Screen Shot 2021-01-10 at 4 44 21 pm

Before

image

@MaxwellBo MaxwellBo requested a review from kumar-ish January 9, 2021 13:49
@cypress
Copy link

cypress bot commented Jan 9, 2021



Test summary

18 0 0 0


Run details

Project Muncoordinated
Status Passed
Commit 10a1a69
Started Jan 12, 2021 6:20 AM
Ended Jan 12, 2021 6:22 AM
Duration 02:03 💡
OS Linux Ubuntu Linux - 16.04
Browser Electron 73

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@MaxwellBo
Copy link
Owner Author

MaxwellBo commented Jan 9, 2021

@kumar-ish fiddle around with this and see if it feels nice. I'm not sure if moving the 'Create account' button to a seperate step is a good thing.

On the one hand, it feels more inline with what people expect, and will allow us to ask questions when accounts are created like display name or marketing consent.

On the other hand, it will slow onboarding down by adding an extra click.

This PR is a good springboard for #105 and #85

@MaxwellBo
Copy link
Owner Author

MaxwellBo commented Jan 9, 2021

Resolved: Fix breaking Cypress tests CypressError: Timed out retrying: Expected to find content: 'Log in' within the element: [ <button.ui.button.nav__auth-status>, 1 more... ] but never did.

@kumar-ish
Copy link
Collaborator

kumar-ish commented Jan 12, 2021

It looks a lot nicer, and feels more intuitive to me; at least this way account creation/login are somewhat separated too. LGTM

I tried amending it without a hacky pixel fix, but we still have to fix this floaty-ass cursed button somehow (and I think it might be beyond my ability to do so):

image

@MaxwellBo
Copy link
Owner Author

I tried amending it without a hacky pixel fix, but we still have to fix this floaty-ass cursed button somehow (and I think it might be beyond my ability to do so):

Yeah, I tried fiddling around with it myself and it seems a little nightmarish :( I considered dropping it entirely and just flat out merging template creation into the setup screen, but I worry that that might negate the whole "oh cool it has committee templates" effect that you immediately see when you start to onboard .

@MaxwellBo MaxwellBo merged commit 513dac9 into master Jan 13, 2021
@kumar-ish kumar-ish deleted the auth-overhaul branch June 12, 2021 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants