[$500] Distance - EReceipt thumbnail is not clickable when distance request is edited offline

[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: 1.4.44-6
Reproducible in staging?: Y
Reproducible in production?: N
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: Applause - Internal Team
Slack conversation:

Action Performed:

1. Go to staging.new.expensify.com
2. Go to workspace chat
3. Create a distance request
4. Go to request details page
5. Go offline
6. Click Distance
7. Add a waypoint and save it
8. Click on the receipt thumbnail

Expected Result:

User will be able to open EReceipt when distance request is edited offline (production behavior)

Actual Result:

EReceipt thumbnail is not clickable when distance request is edited offline

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

Bug6394556_1709063187268.20240227_204810.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01e647aab2a32331ae
• Upwork Job ID: 1762569075317325824
• Last Price Increase: 2024-02-27
• Automatic offers:
• paultsimura | Reviewer | 0
• shubham1206agra | Reviewer | 0
• aimane-chnaif | Contributor | 0

[melvin-bot]

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

[melvin-bot]

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

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[melvin-bot]

Triggered auto assignment to @thienlnam (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[lanitochka17]

We think that this bug might be related to #wave5
CC @dylanexpensify

[paultsimura]

It's caused by #34133.
@shubham1206agra could you please take a look? We should make the ConfirmedRoute clickable when using it here:

App/src/components/ReportActionItem/MoneyRequestView.tsx

Lines 256 to 258 in 069efef

	{showMapAsImage ? (
	<ConfirmedRoute transaction={transaction} />
	) : (

[melvin-bot]

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

Offer link
Upwork job

[shubham1206agra]

@paultsimura This still comes under the regression period, as it was reported under that period. But if @thienlnam agrees, I would like @paultsimura to be a contributor here instead, as he has enough context about the issue and solution. And either @aimane-chnaif or I can be a C+ here instead.

[paultsimura]

This still comes under the regression period, as it was reported under that period.

That's why I wanted to double-check because this issue's been open for quite a while already.
If it won't be fixed as a regression, I think it would be fair to go through a regular External process allowing Contributors to post proposals as usual.

[thienlnam]

Since this comes directly from a PR and was opened within the time period, let's have that handled as a regression. We wouldn't issue payment for this so please handle it with @aimane-chnaif.

[paultsimura]

Ok, unassigning myself 👍

[aimane-chnaif]

Waiting for @shubham1206agra to raise follow-up PR

[melvin-bot]

@thienlnam, @shubham1206agra, @aimane-chnaif Eep! 4 days overdue now. Issues have feelings too...

[aimane-chnaif]

Sorry I wanted to review PR but had to unassign as I'll be OOO from tomorrow

[paultsimura]

Can I be assigned back as a C+ here?

[thienlnam]

Going to adjust to a monthly since this is not urgent

[thienlnam]

This is an edge case and not a priority - going to just close it