Automattic / wp-calypso

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

Stepper: Roll out explicit login requirements to all signup Stepper flows [signup only] #92291

Closed gabrielcaires closed 2 months ago

gabrielcaires commented 3 months ago

Details

Copied from https://github.com/Automattic/dotcom-forge/issues/7643 to simplify the management.

This issue is intended to track the work to roll out the changes from https://github.com/Automattic/wp-calypso/pull/91241 to other flows. The core change in that PR is to specify whether individual Stepper steps should require a login and move more logic around login redirects into the Stepper framework.

Done is: All Stepper flows are using the new requiresLoggedInUser flag on steps where users need to be logged in

Checklist

No response

Related

No response

### Tasks
- [ ] https://github.com/Automattic/wp-calypso/pull/91636
- [ ] https://github.com/Automattic/wp-calypso/pull/92136
- [ ] https://github.com/Automattic/wp-calypso/pull/92126
- [x] Site-migration: Automattic/wp-calypso#91241
- [x] Hosted-site-migration  Automattic/wp-calypso#91241
- [x] ecommerce #91612
- [x] free #92292
- [x] design-first #92296
- [x] start-writing #92293
- [x] Assembler-first #92295
- [x] entrepreneur #92298
- [x] newsletter #92340
- [x] sensei #92299
- [x] import-hosted-site #92329
- [x] videopress --  #92393
- [x] LINK_IN_BIO_TLD_FLOW #92330
- [x] new-hosted-site-user-included -- Using completly different strategy
- [x] readymade-template #92395
- [x] hundred-year-plan #92335
- [x] link-in-bio #92337
- [x] reblogging  #92338
- [x] Ai Assembler #92339
gabrielcaires commented 3 months ago

@daledupreez I created a task for each flow using isSignupFlow: true. The list is organized based on the most frequently used flow to the least used flow.