[HOLD for Payment 2024-08-22][$250] Search - Expense RHP closes after clicking on the From subtitle link

[lanitochka17]

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: 9.0.14-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Action Performed:

1. Go to staging.new.expensify.com
2. Go to any chat
3. Submit an expense
4. Go to Search
5. Click on the expense submitted in Step 3
6. Click on the RHP header
7. Click on the From subtitle link

Expected Result:

App will open the report from the subtitle link

Actual Result:

The expense RHP closes after clicking on the From subtitle link

Workaround:

Unknown

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

Add any screenshot/video evidence

Bug6557025_1722319302164.20240730_135933.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01d7addf8985df968d
• Upwork Job ID: 1818576628459044491
• Last Price Increase: 2024-07-31
• Automatic offers:
• nkdengineer | Contributor | 103461487

Issue Owner

Current Issue Owner: @fedirjh

[melvin-bot]

Triggered auto assignment to @jliexpensify (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.

[lanitochka17]

@jliexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[nkdengineer]

Proposal

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

The expense RHP closes after clicking on the From subtitle link

What is the root cause of that problem?

When we open the report detail page from RHP Report Screen and click on the parent subtitle, we use goBack to navigate back to the report screen but it doesn't work as expected.

App/src/components/ParentNavigationSubtitle.tsx

Lines 44 to 47 in 8e79c59

	Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID));
	if (isVisibleAction && !isOffline) {
	// Pop the chat report screen before navigating to the linked report action.
	Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID, parentReportActionID));

We only navigate back to the fallbackRoute if isFirstRouteInNavigator is true or we're in the central panel. In this case, the RHP stack has two screens RHP ReportScreen --> Report Detail Page so isFirstRouteInNavigator is false at both times we call goBack (we call goBack two times immediately so navigationRef.current.getState() is the same) and then we only go back to the previous screen two times Report Detail Page --> RHP ReportScreen --> Search Page.

App/src/libs/Navigation/Navigation.ts

Line 211 in 8e79c59

const isFirstRouteInNavigator = !getActiveRouteIndex(navigationRef.current.getState());

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

We should pass shouldEnforceFallback param as true here to make sure we will use the fallbackRoute when we call goBack the second time.

App/src/components/ParentNavigationSubtitle.tsx

Line 47 in 8e79c59

Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID, parentReportActionID));

What alternative solutions did you explore? (Optional)

We can use Navigation.isNavigationReady() to wait until the first goBack is complete before calling goBack the second time. With this at the second time the RHP stack has only RHP ReportScreen and then isFirstRouteInNavigator is true that leads we navigate to the fallbackRoute

// Pop the thread report screen before navigating to the chat report.
Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID));
if (isVisibleAction && !isOffline) {
    // Pop the chat report screen before navigating to the linked report action.
    Navigation.isNavigationReady().then(() => {
        Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID, parentReportActionID));
    });
}

App/src/components/ParentNavigationSubtitle.tsx

Lines 44 to 47 in 8e79c59

	Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID));
	if (isVisibleAction && !isOffline) {
	// Pop the chat report screen before navigating to the linked report action.
	Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReportID, parentReportActionID));

[melvin-bot]

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

[jliexpensify]

I'm able to repro this

[melvin-bot]

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

[fedirjh]

I am unable to replicate in the latest main

CleanShot.2024-08-01.at.12.08.30.mp4

[nkdengineer]

@fedirjh I'm still able to reproduce this on the latest main bug by disabling the strict mode. Correct me if I missed something.

Screen.Recording.2024-08-01.at.20.24.02.mov

[melvin-bot]

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

[fedirjh]

Thank you, @nkdengineer, for the suggestion. I managed to reproduce the issue on the dev by disabling strict mode. The proposed solution also seems satisfactory to me, so let's move forward with implementing the main solution.

Let's proceed with @nkdengineer's proposal;

🎀 👀 🎀 C+ reviewed

[melvin-bot]

Triggered auto assignment to @NikkiWines, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[nkdengineer]

@NikkiWines Please help to take a look when you have a chance, thanks.

[NikkiWines]

Sorry for the delay, agreed that @nkdengineer's proposal looks good 👍

[melvin-bot]

📣 @nkdengineer 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[fedirjh]

BugZero Checklist:

• Link to the PR: N/A, It seems this bug was hidden for a long time and has been discovered recently when a new feature was released.
• Link to comment: N/A No PR.
• Link to discussion: NA, With the strict mode enabled, it should be impossible to catch this bug in a dev environment.
• Determine if we should create a regression test for this bug: Yes

---

Regression Test Proposal

Submit an expense
Go to Search
Click on the expense submitted in Step 3
Click on the RHP header
Click on the From subtitle link
Verify that: App will open the report from the subtitle link

• Do we agree 👍 or 👎

[jliexpensify]

Paying now, apologies for the delay!

• C: @nkdengineer $250
• C+: @fedirjh $250 (ND Payment)

Upwork job

[jliexpensify]

Paid and job remvoed

[garrettmknight]

$250 approved for @fedirjh