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

We created multiple workspaces for user in onboarding #52519

Open
danielrvidal opened this issue Nov 14, 2024 · 9 comments
Open

We created multiple workspaces for user in onboarding #52519

danielrvidal opened this issue Nov 14, 2024 · 9 comments
Assignees
Labels
Daily KSv2

Comments

@danielrvidal
Copy link
Contributor

This is being discussed here so I'm just creating a tracking issue for us to get to the bottom of it with @marcaaron @trjExpensify

https://expensify.slack.com/archives/C07HPDRELLD/p1731537294677189

@marcaaron
Copy link
Contributor

I think @carlosmiceli is working on moving the policy creation to the CreateAccount step. I am wondering if this is at all related to those changes. At least, it seems like it will be fixed by them since we will no longer be creating the policy via an API call triggered in App.

@trjExpensify
Copy link
Contributor

Interesting. Carlos, how does that work if you create an account on new.expensify.com directly where there's no signUpQualifiers to choose? 🤔

@carlosmiceli
Copy link
Contributor

What @marcaaron says is correct, that's what I'm working on. @trjExpensify so far, we would just not create a workspace for any signup that hasn't selected a qualifier. In fact, creating an account will not fail if for some reason the workspace fails to get created. But let's check with @anmurali to see if we'd like to add/change some logic about it.

@trjExpensify
Copy link
Contributor

@trjExpensify so far, we would just not create a workspace for any signup that hasn't selected a qualifier

Okay, which means they would then see the onboarding modal and selecting the manage my team path would create a workspace?

@carlosmiceli
Copy link
Contributor

Oh, that's a good question (and sounds like a good idea) but I don't know about that being discussed anywhere and it's definitely outside of the scope of the issue I'm working on (which is all BE and happening before ND is loaded).

@trjExpensify
Copy link
Contributor

Sorry, maybe I'm not explaining myself well.

  1. You said you're working on creating a workspace after you choose this on the expensify.com home page:
image
  1. Today, the timing of creating the workspace is after you click this:
image

Q: In the expensify.com sign-up flow, how are you avoiding creating two workspaces on sign-up? Are you skipping that "How many employees do you have?" screen, and as a result, not triggering the CreateWorkspace command a second time?

Q: If you sign-up via new.expensify.com where there are no signUpQualifiers to select on the home page, will you still see the "How many employees do you have?" screen if you choose "Manage my team's expenses" in the first step, and thus still trigger CreateWorkspace at that point like today?

image image image

@carlosmiceli
Copy link
Contributor

In the expensify.com sign-up flow, how are you avoiding creating two workspaces on sign-up? Are you skipping that "How many employees do you have?" screen, and as a result, not triggering the CreateWorkspace command a second time?

I haven't got to this point yet, but I think as Marc said, we're going to remove the current CreateWorkspace logic of creating the policy in App during onboarding (at least temporarily so we don't create duplicates).

If you sign-up via new.expensify.com where there are no signUpQualifiers to select on the home page, will you still see the "How many employees do you have?" screen if you choose "Manage my team's expenses" in the first step, and thus still trigger CreateWorkspace at that point like today?

To clarify, I'm not working in anything that would change any logic that happens after/during signup through new.expensify.com, other than replacing the logic as I mentioned above. You will see whatever you see while signing up now. This means that no workspace will get created when signing up through new.expensify.com until we actually want to implement some logic around that.

I hope I'm understanding the question 😅

@trjExpensify
Copy link
Contributor

trjExpensify commented Nov 15, 2024

Got it, so yeah.. if you remove the logic to create a workspace in the onboarding modal completely, you're going to break creating a workspace during onboarding for:

  • a sign-up on new.expensify.com on web/mWeb
  • a sign-up on the New Expensify mobile app (we're going to deprecate it soon, but still worth noting)
  • a sign-up on the native Expensify mobile app because that mobile home page also does not have signUpQualifiers (we're about to start defaulting new account sign-ups in NewDot after the next release that's on the way out the door)

So I don't think we should remove it completely. Rather, if you come from expensify.com or use.expensify.com and a workspace is created for you because of the signUpQualifier selection, we skip the "How many employees do you have?" screen and creating a workspace on completion of the onboarding step. If you don't have a signUpQualifier selection, we show it and create a workspace like today.

@melvin-bot melvin-bot bot added the Overdue label Nov 18, 2024
@carlosmiceli
Copy link
Contributor

Ah, gotcha! Yup, that sounds good, we can do that. By the way, while it's an edge case, we should consider this again then. I think with this new situation, it's justified to just hide the back button instead of having to implement more complicated logic to skip the workspace creation if they happen to go back.

@carlosmiceli carlosmiceli added Daily KSv2 and removed Daily KSv2 labels Nov 19, 2024
@melvin-bot melvin-bot bot removed the Overdue label Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2
Projects
Status: No status
Development

No branches or pull requests

4 participants