[HOLD #11768] [$1000] Hide create a workspace page when app is loading

[chiragsalian]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

1. Go to http://localhost:8080/settings/workspaces in the URL while being logged out.
2. Log into the app.

Expected Result:

Workspace should show a loader and show workspaces when ready

Actual Result:

The create a workspace screen flashes for a second before showing the workspaces

Screen.Recording.2022-10-21.at.08.43.21.mov

Workaround:

Can the user still use Expensify without this being fixed? - Yes. this is a polish to improve experience.

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number:
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01b610ee831855a28a
• Upwork Job ID: 1636765010047090688
• Last Price Increase: 2023-03-17

[chiragsalian]

Low priority so haven't gotten to it yet

[chiragsalian]

Low priority so haven't gotten to it yet

[chiragsalian]

Low priority so haven't gotten to it yet

[mountiny]

@chiragsalian would you be interested on outsourcing this to @narefyev91 from Callstack?

[Milos213]

Proposal

Please re-state the problem that we are trying to solve in this issue.

Empty workspace page shows before loading workspaces after login

What is the root cause of that problem?

App/src/pages/workspace/WorkspacesListPage.js

Lines 185 to 195 in bcdcfe5

	{_.isEmpty(workspaces) ? (
	<BlockingView
	icon={Expensicons.Building}
	title={this.props.translate('workspace.emptyWorkspace.title')}
	subtitle={this.props.translate('workspace.emptyWorkspace.subtitle')}
	/>
	) : (
	<ScrollView style={styles.flex1}>
	{_.map(workspaces, (item, index) => this.getMenuItem(item, index))}
	</ScrollView>
	)}

If workspaces are empty (no matter really empty or not loaded yet), we always show BlockingView with empty workspace title and subtitle.

What changes do you think we should make in order to solve the problem?

We can early return FullScreenLoadingIndicator in render() if _.isEmpty(this.props.policies) = true.
This condition should be enough to check if workspaces are loaded or not because at least 1 policy (Concierge) should exist for created account.
This is working well even for new accounts.

[MelvinBot]

📣 @Milos213! 📣

Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
2. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
3. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[MelvinBot]

@chiragsalian Whoops! This issue is 2 days overdue. Let's get this updated quick!

[MelvinBot]

@chiragsalian Huh... This is 4 days overdue. Who can take care of this?

[chiragsalian]

whoops, my bad, i lost track of this one. I should have made this external ages ago. Just briefly tested and its still an issue.

Unassigning myself since I'm OOO for a bit but it should be great for someone external to address. Thanks for the proposal @Milos213 but I'll let the assigned C+ and engineer review it.

[MelvinBot]

Job added to Upwork: https://www.upwork.com/jobs/~01b610ee831855a28a

[MelvinBot]

Triggered auto assignment to @adelekennedy (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

[s77rt]

Same ^

[MelvinBot]

@s77rt, @Gonals, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!

[MelvinBot]

@s77rt, @Gonals, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!

[s77rt]

Still on hold

[adelekennedy]

on hold

[Gonals]

@MariaHCD, tagging you on this one as I'm going on parental leave! Thanks!

[MelvinBot]

@MariaHCD, @s77rt, @adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[MariaHCD]

Looks like we're holding on the navigation reboot. Downgrading priority to weekly!

[adelekennedy]

@MariaHCD should we downgrade this further? This issues been open for 6+ months - I'm not sure if we should downgrade, close or keep it as is until we resolve the navigation issues

[MariaHCD]

I see there's progress being made on the navigation reboot so let's downgrade the priority further and pick this back up again.

[s77rt]

Still on hold.

[mountiny]

This is fixed by the refactor, there is no reporter so I am closing this