Comment by winrid

Nice! However, none of this is required for this endpoint. Here's why:

1. The connect action could be replaced by doing the connection once on app startup.

2. The validation could be replaced with middleware like express-joi.

3. The stripe/email steps should be asynchronous (ex: simple crons). This way, you create the user and that's it. If Stripe is down, or the email provider is down, you still create the user. If the server restarts while someone calls the endpoint, you don't end up with a user with invalid Stripe config. You just create a user with stripeSetup=false and welcomeEmailSent=false and have some crons that every 5 seconds query for these users and do their work. Also, ensure you query for false and not "not equal to true" here as it's not efficient.