Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.38k stars 1.97k forks source link

Remove email verification step when onboarding Woo Express #84976

Closed adrianduffell closed 6 months ago

adrianduffell commented 7 months ago

User Story

As a new Woo Express user, I want to sign-up quickly, so that I can start creating my store.

Description

In the Woo Express sign-up flow, new users that create a WordPress.com account are required to verify their email. This screen should be updated with Woo branding.

Email verification

Figma

The screen presently looks like:

image

chihsuan commented 7 months ago

This is a bit tricky to customize the screen because the users are redirected to this page by Jetpack SSO module when visiting the created new site, not Calypso. It's possible that we can set a cookie or local storage to preserve the woo context before the page.

However, I think a better solution is to implement the magic link signup for Woo Express flow.

chihsuan commented 6 months ago

Adrian and I discussed this in our 1:1 today. We want to fix "Email verification screen" in January.

However, I think a better solution is to implement the magic link signup for Woo Express flow.

Because we are not likely to have a passwordless signup for Woo Express flow in January, Adrian mentioned that we can probably do a quick fix by skipping the verification step for new users when logging Woo Express free trial sites.

I dug a little deeper today and found that the control logic is actually in WPCOM.

So it seems that we can simply add a few lines to remove email verification for free trial users. I would prefer this approach if it's legal to skip email verification because it's just a temporary solution before launching passwordless sign up.

Otherwise, adding a Woo verification screen and redirecting users to it is possible, but that would be a bit more effort.

What do you think? @adrianduffell cc @Automattic/ghidorah

adrianduffell commented 6 months ago

if it's legal to skip email verification

Thanks for your work looking into this, @chihsuan! I'll follow-up about this to help us make a decision.

adrianduffell commented 6 months ago

So it seems that we can simply add a few lines to remove email verification for free trial users. I would prefer this approach if it's legal to skip email verification because it's just a temporary solution before launching passwordless sign up.

Confirmed it's legal to skip email verification so let's go ahead with this approach as a temporary solution.

chihsuan commented 6 months ago

D133829-code

p-jackson commented 5 months ago

@chihsuan I've just deployed a change to Jetpack SSO that I think might supersede this improvement. Discussed here p1HpG7-qll-p2, it solves the issue of email verification for the general case which and I think it should cover the situation described by OP.

Check out the code change here D133828-code, you might be able to remove the Woo Express-specific work around.

chihsuan commented 5 months ago

@p-jackson Nice, thanks for letting me know! 👍 I'll check out the code.