[$500] Clicking the back button on the Merchant page for an expense takes to a blank page with just the LHN

[m-natarajan]

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: 1.4.37-1
Reproducible in staging?: y
Reproducible in production?: y
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: @yuwenmemon
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707249292907579

Action Performed:

1. Smartscan a receipt in a workspace chat
2. Open the expense report details page
3. Click on the merchant field
4. Click on back button

Expected Result:

Should land in the expense details page

Actual Result:

Blank page opens

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

Recording.2700.mp4

Kapture.2024-02-06.at.11.53.39.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0199b6cdb65097259b
• Upwork Job ID: 1754982516276056064
• Last Price Increase: 2024-02-06

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to @MitchExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[melvin-bot]

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

[muas19]

Proposal

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

Clicking the back button on the Merchant page for an expense takes to a blank page with just the LHN

What is the root cause of that problem?

We are not passing the onBackButtonPress prop to the HeaderWithBackButton component.

App/src/pages/EditRequestMerchantPage.js

Line 50 in ccad061

<HeaderWithBackButton title={translate('common.merchant')} />

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

Pass the prop with Navigation.dismissModal()
<HeaderWithBackButton onBackButtonPress={() => Navigation.dismissModal()} title={translate('common.merchant')} />

What alternative solutions did you explore? (Optional)

n/a

[neonbhai]

Proposal

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

Clicking the back button on the Merchant page for an expense takes to a blank page with just the LHN

What is the root cause of that problem?

We are missing the back button logic in EditRequestMerchantPage

App/src/pages/EditRequestMerchantPage.js

Line 50 in ccad061

<HeaderWithBackButton title={translate('common.merchant')} />

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

We shoud pass onBackButtonPress={Navigation.goBack} as the prop, like we do in similar pages.

[muas19]

#35965 (comment) Updated with formatting and typo.

[allgandalf]

Maybe related to: #35626

[trjExpensify]

@adamgrzybowski is going to sense check whether his solution on the main issue addressing goBack bugs will fix this. Let's hold on making a PR until then!

[adamgrzybowski]

@trjExpensify should be fixed by #36050

[hayata-suenaga]

The issue is tracked in #35626. Closing this in favor of that one 🙇