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

[Awaiting Payment 20th May] [Guided Setup Stage 1] Ask for business name before asking for display name #41589

Closed
6 tasks done
trjExpensify opened this issue May 3, 2024 · 12 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor

Comments

@trjExpensify
Copy link
Contributor

trjExpensify commented May 3, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: v1.4.70-1
Reproducible in staging?: Y
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @trjExpensify
Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1714746065450189?thread_ts=1714400621.865599&cid=C036QM0SLJK

Action Performed:

  1. Sign up with a new account
  2. Click Manage my team's Expenses in the onboarding modal
  3. Observe the next question you're asked is for your display name
  4. Enter your display name
  5. Observe the next question you're asked is for your business name

Expected Result:

When a new user selects the Manage my team's expenses onboarding intent:

  1. The next question we ask should be for their Business name
  2. Upon clicking the confirmation button on the business name page we should trigger the CreateWorkspace API command immediately

Actual Result:

We ask for business name last, so as a result, we trigger the CreateWorkspace command at the end of the onboarding flow.

Workaround:

N/A

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

2024-05-03_15-23-09.mp4

View all open jobs on GitHub

CC: @anmurali @francoisl @NikkiWines @mountiny @danielrvidal

Issue OwnerCurrent Issue Owner: @thesahindia
@trjExpensify trjExpensify added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels May 3, 2024
@trjExpensify trjExpensify self-assigned this May 3, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 3, 2024
Copy link

melvin-bot bot commented May 3, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External)

Copy link

melvin-bot bot commented May 3, 2024

Current assignee @trjExpensify is eligible for the Bug assigner, not assigning anyone new.

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label May 3, 2024
@DylanDylann DylanDylann mentioned this issue May 3, 2024
50 tasks
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels May 3, 2024
@ishpaul777
Copy link
Contributor

ishpaul777 commented May 3, 2024

When a new user selects the Manage my team's expenses onboarding intent:

  • The next question we ask should be for their Business name
  • Upon clicking the confirmation button on the business name page we should trigger the CreateWorkspace API command immediately

@trjExpensify Is this behaviour specific to "Manage my team's expenses" option or for all other options?

@trjExpensify
Copy link
Contributor Author

Yes, just that option. None of the other options ask for a business name. 👍

@ishpaul777
Copy link
Contributor

ishpaul777 commented May 3, 2024

ohk, i asked becuase with current PR changes it was asking for bussiness name for all options

@trjExpensify
Copy link
Contributor Author

Yeah, I see the commit to not do that 😅

Copy link

melvin-bot bot commented May 8, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@DylanDylann
Copy link
Contributor

@trjExpensify The PR was deployed to production 2 weeks ago. Could you help to move forward with this issue

@trjExpensify
Copy link
Contributor Author

Ah, the automation didn't run. Annoying! Payment summary as follows:

Offers sent!

@trjExpensify trjExpensify added Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 and removed Reviewing Has a PR in review Weekly KSv2 labels May 20, 2024
@trjExpensify trjExpensify changed the title [Guided Setup Stage 1] Ask for business name before asking for display name [Awaiting Payment 20th May] [Guided Setup Stage 1] Ask for business name before asking for display name May 20, 2024
@ishpaul777
Copy link
Contributor

Thanks! offer accepted

@trjExpensify
Copy link
Contributor Author

@ishpaul777, paid!

@trjExpensify
Copy link
Contributor Author

@DylanDylann paid! Closing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor
Projects
No open projects
Archived in project
Development

No branches or pull requests

5 participants