[$250] Workspace - No pending workspace join is shown via share link

[IuliiaHerets]

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

---

Version Number: 9.0.60-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue was found when executing this PR: #51631
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

Precondition: user A with a workspace.

1. User A: go to Settings > Workspaces > workspace > Profile
2. Click on Share > Copy URL
3. Send URL to user B, that is not a member of this ws
4. User B (logged in ND in mWeb): open URL in mobile browser

Expected Result:

List of workspaces opened.
Workspace user B was sent a link has a label 'Requested'.

Actual Result:

Settings tab is opened. Sometimes, tap on Workspaces has no response. When open link again, user B is joined ws automatically.

Workaround:

Unknown

Platforms:

• Android: Standalone
• Android: HybridApp
• Android: mWeb Chrome
• iOS: Standalone
• iOS: HybridApp
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Bug6662227_1731398893074.Screenrecorder-2024-11-11-22-14-11-219.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021856489377948710082
• Upwork Job ID: 1856489377948710082
• Last Price Increase: 2024-12-25

Issue Owner

Current Issue Owner: @hoangzinh

[melvin-bot]

Triggered auto assignment to @kadiealexander (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[melvin-bot]

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

[melvin-bot]

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

[ikevin127]

This is a tricky one as it seems to be Android (mWeb only ?) specific, possibly something navigation related because the logic below works as expected on all other platforms:

App/src/pages/workspace/WorkspaceJoinUserPage.tsx

Lines 29 to 75 in 99da1ac

	function WorkspaceJoinUserPage({route, policy}: WorkspaceJoinUserPageProps) {
	const styles = useThemeStyles();
	const policyID = route?.params?.policyID;
	const inviterEmail = route?.params?.email;
	const isUnmounted = useRef(false);
	useEffect(() => {
	if (!isJoinLinkUsed) {
	return;
	}
	navigateAfterJoinRequest();
	}, []);
	useEffect(() => {
	if (isUnmounted.current || isJoinLinkUsed) {
	return;
	}
	if (!isEmptyObject(policy) && !policy?.isJoinRequestPending && !PolicyUtils.isPendingDeletePolicy(policy)) {
	Navigation.isNavigationReady().then(() => {
	Navigation.goBack(undefined, false, true);
	Navigation.navigate(ROUTES.WORKSPACE_INITIAL.getRoute(policyID ?? '-1'));
	});
	return;
	}
	MemberAction.inviteMemberToWorkspace(policyID, inviterEmail);
	isJoinLinkUsed = true;
	Navigation.isNavigationReady().then(() => {
	if (isUnmounted.current) {
	return;
	}
	navigateAfterJoinRequest();
	});
	}, [policy, policyID, inviterEmail]);
	useEffect(
	() => () => {
	isUnmounted.current = true;
	},
	[],
	);
	return (
	<ScreenWrapper testID={WorkspaceJoinUserPage.displayName}>
	<FullScreenLoadingIndicator style={[styles.flex1, styles.pRelative]} />
	</ScreenWrapper>
	);
	}

[Shahidullah-Muffakir]

Proposal

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

Requested badge is not showing in mobile devices

What is the root cause of that problem?

we are removing the requested badge in mobile devices here:

App/src/pages/workspace/WorkspacesListRow.tsx

Line 141 in 99da1ac

<View style={[styles.flexRow, !shouldUseNarrowLayout && styles.workspaceThreeDotMenu]}>

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

if we want to show it in mobile devices remove the !shouldUseNarrowLayout

[Shahidullah-Muffakir]

[hoangzinh]

Hi @Shahidullah-Muffakir please note that it's not only "Requested badge" issue but also navigation issue in small screen devices. Can you retake a look?

[FitseTLT]

Proposal

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

Workspace - No pending workspace join is shown via share link

What is the root cause of that problem?

Here we have two consecutive navigation to settings and workspace settings pages for small screen web after goBack

App/src/libs/navigateAfterJoinRequest/index.web.ts

Lines 6 to 10 in 09f6db7

	Navigation.goBack(undefined, false, true);
	if (getIsSmallScreenWidth()) {
	Navigation.navigate(ROUTES.SETTINGS);
	}
	Navigation.navigate(ROUTES.SETTINGS_WORKSPACES);

we do that to put settings page below workspace settings page in the stack but for only android mWeb this is making the app stuck.

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

We can fix it if we join the goBack and the navigation to settings page together to achieve the same result

navigateAfterJoinRequest = () => {
    const isSmallScreen = getIsSmallScreenWidth();
    Navigation.goBack(isSmallScreen ? ROUTES.SETTINGS : undefined, isSmallScreen, true);

    Navigation.navigate(ROUTES.SETTINGS_WORKSPACES);

POC:

2024-11-15.19-51-17.mp4

What alternative solutions did you explore? (Optional)

[melvin-bot]

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

[jjcoffee]

Proposal

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

On Android mWeb only, opening the workspace join link URL results in an incorrect navigation state where the settings page is showing with the Workspaces menu item selected (as if it were in the desktop view), but the Workspaces screen is not showing.

What is the root cause of that problem?

We perform multiple navigations to get to the final settings/workspaces route, which is currently triggered after isNavigationReady here:

App/src/pages/workspace/WorkspaceJoinUserPage.tsx

Lines 55 to 60 in 99da1ac

	Navigation.isNavigationReady().then(() => {
	if (isUnmounted.current) {
	return;
	}
	navigateAfterJoinRequest();
	});

When we navigate directly via URL to the join link, it seems that there's some sort of race condition in the way Chrome handles navigation transitions (interestingly I can only reproduce on Android Chrome and Linux Chrome, but not MacOS Chrome). Most likely the navigation transitions are being queued differently in some subtle way.

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

To ensure that we wait until navigation transitions complete first, we can wrap the navigation with runAfterInteractions to ensure the interaction queue is idle, which reduces the chance of intermediate navigation steps being skipped or misordered.

InteractionManager.runAfterInteractions(() => {
	navigateAfterJoinRequest();
});

What alternative solutions did you explore? (Optional)

Whilst digging into if there was some general navigation issue, I noticed that this call to updateLastVisitedPath seems to be playing a part in the issue too:

App/src/libs/Navigation/NavigationRoot.tsx

Lines 61 to 66 in 7c90011

	if (focusedRoute && !CONST.EXCLUDE_FROM_LAST_VISITED_PATH.includes(focusedRoute?.name)) {
	updateLastVisitedPath(currentPath);
	if (currentPath.startsWith(`/${ROUTES.ONBOARDING_ROOT.route}`)) {
	updateOnboardingLastVisitedPath(currentPath);
	}
	}

Therefore adding WORKSPACE_JOIN_USER to EXCLUDE_FROM_LAST_VISITED_PATH also works as a fix and I don't see an obvious downside as we wouldn't want/need to return to the join link. This effectively bypasses the issue, but it's 50/50 on whether this counts as a workaround or not 😄

[hoangzinh]

Hi, @FitseTLT @jjcoffee Do you still reproduce this issue in the latest main branch? I tried a few times, but it seems everything works for me

Screen.Recording.2024-11-19.at.22.11.25.mov

[jjcoffee]

@hoangzinh I had slightly inconsistent behaviour, sometimes the first load works fine, but all subsequent loads wouldn't.

[FitseTLT]

@hoangzinh Yes I can consistently reproduce it

2024-11-19.20-00-49.mp4

[hoangzinh]

I'm unable to reproduce this issue. @jjcoffee if you can reproduce this issue, would you like to be a C+ on this issue?

Screen.Recording.2024-11-20.at.21.49.25.mov

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[hoangzinh]

Same as above #52394 (comment)

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[hoangzinh]

@kadiealexander should we adjust bounty to get more eyes on this issue?

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[melvin-bot]

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

[melvin-bot]

@hoangzinh @kadiealexander this issue is now 4 weeks old, please consider:

• Finding a contributor to fix the bug
• Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
• If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

[melvin-bot]

@hoangzinh, @kadiealexander Huh... This is 4 days overdue. Who can take care of this?

[hoangzinh]

Issue not reproducible during KI retests. (First week)

@kadiealexander can you request to test this issue again? It's tricky for me to reproduce this issue so I'm hard to verify whether it's reproducible or not.

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[mvtglobally]

Issue not reproducible during KI retests. (Second week)

[hoangzinh]

@kadiealexander I think we can close this issue. If anyone can reproduce this issue again, we can re-open it later.

[jjcoffee]

I am still able to consistently reproduce it on latest main:

android-chrome-2024-12-16_16.27.13.mp4

[hoangzinh]

Thanks @jjcoffee

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

[melvin-bot]

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

[jjcoffee]

@hoangzinh Should I take this over since I can reproduce? I think we should maybe get one of the agencies to take a look.

[hoangzinh]

I am happy to let you take it @jjcoffee. cc @kadiealexander can you help to assign this issue to @jjcoffee. I'm hard to reproduce this issue.

[melvin-bot]

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

[melvin-bot]

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸