[$250] iOS - IOU - "Not here" error appears briefly after deleting expense

[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.43-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): andreylazutkinutest+02102@gmail.com
Issue reported by: Applause - Internal Team

Issue found when executing PR #49878

Action Performed:

1. Open the app
2. Log in with a Gmail account
3. Create a new workspace
4. Open the workspace chat
5. Submit a manual expense with any amount and merchant
6. Open the expense
7. Set the "Date" to a future one
8. Tap on the expense header
9. Delete the expense

Expected Result:

I should be able to delete it without any errors

Actual Result:

"Not here" error appears briefly after chenging expense date and deleting it after

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

Bug6622222_1727881924347.HABH7257.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021841519823485482301
• Upwork Job ID: 1841519823485482301
• Last Price Increase: 2024-10-09

Issue Owner

Current Issue Owner: @getusha

[melvin-bot]

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

[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.

[tgolen]

I don't think this needs to be a blocker. I cannot find anything listed in the deploy checklist that stands out to me, so I am going to move this to be external and get some help trying to find what caused it and a solution.

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

[huult]

Edited by proposal-police: This proposal was edited at 2024-10-03 04:45:01 UTC.

Proposal

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

"Not here" error appears briefly after deleting expense

What is the root cause of that problem?

After deleting the expense, this report will be set to null.

App/src/pages/ReportDetailsPage.tsx

Line 719 in e61ee7f

const deleteTransaction = useCallback(() => {

And after calling the delete expense function, it will navigate back to the previous screen, where the report returns undefined because it was set in the IOU.deleteMoneyRequest function. As a result, the 'not here' page appears.

App/src/pages/home/ReportScreen.tsx

Lines 270 to 276 in e61ee7f

	useEffect(() => {
	if (!report?.reportID || shouldHideReport) {
	wasReportAccessibleRef.current = false;
	return;
	}
	wasReportAccessibleRef.current = true;
	}, [shouldHideReport, report]);

The condition shows the 'Not here' page in this issue

App/src/pages/home/ReportScreen.tsx

Line 392 in e61ee7f

if (!wasReportAccessibleRef.current && !firstRenderRef.current && !reportID && !isOptimisticDelete && !reportMetadata?.isLoadingInitialReportActions && !userLeavingStatus) {

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

We prevent updating wasReportAccessibleRef after completing the navigation back to avoid showing the 'Not Here' page, something like this:

// .src/pages/home/ReportScreen.tsx#L270
    useEffect(() => {
+        InteractionManager.runAfterInteractions(() => {
+            requestAnimationFrame(() => {
                if (!report?.reportID || shouldHideReport) {
                    wasReportAccessibleRef.current = false;
                    return;
                }
                wasReportAccessibleRef.current = true;
+            });
+        });
    }, [shouldHideReport, report]);

Or we could delay navigating back in onModalHide

POC

Screen.Recording.2024-10-03.at.11.16.58.mov

REPRODUCED THIS ISSUE:

Screen.Recording.2024-10-03.at.11.18.21.mov

[mkzie2]

Proposal

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

"Not here" error appears briefly after chenging expense date and deleting it after

What is the root cause of that problem?

• The issue ""Hmm, not there" briefly is shown while navigating to the IOU preview" was already fixed in PR.

• But after PR, we encountered that bug again. It is because of:

App/src/pages/home/ReportScreen.tsx

Line 272 in e61ee7f

wasReportAccessibleRef.current = false;

where we set wasReportAccessibleRef.current to false althought it was true before.

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

• We should remove:

App/src/pages/home/ReportScreen.tsx

Line 272 in e61ee7f

wasReportAccessibleRef.current = false;

• I tested it, and after removing the mentioned line, the bug that the line wasReportAccessibleRef.current = false was intended to fix is no longer present.

What alternative solutions did you explore? (Optional)

[isabelastisser]

@getusha, please review the proposals above. Thanks!

[getusha]

@mkzie2 i am still able to reproduce the issue after applying your solution

Screen.Recording.2024-10-06.at.8.27.54.in.the.morning.mov

[getusha]

@huult your solution works well, can we pin point why this issue resurfaced? perhaps #36477 caused this?

[mkzie2]

@getusha Can you provide steps to reproduce the bug, or try refreshing your browser to ensure the changes are applied? I tested the solution on my end, and it works as expected. Here’s a slowed-down version to demonstrate: after deleting an expense, the skeleton page appears instead of the 'Not here' page:

Screen.Recording.2024-10-06.at.13.18.10.mov

[huult]

Hi @getusha ,
Yes, I tried reverting the PR, but the issue didn't occur as a result.

POC

Screen.Recording.2024-10-06.at.21.28.53.mov

[mkzie2]

@getusha Just a heads-up, my solution effectively reverts the PR that caused this bug. Someone had flagged the issue in this comment, but C+ couldn't reproduce it at the time.

[isabelastisser]

@getusha, can you please provide an update? Thanks!

[melvin-bot]

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

[getusha]

@mkzie2 just follow the steps in this issue, i am still able to reproduce it with your solution just like @huult did #50085 (comment)

[getusha]

@huult could you track down which PR reintroduced this issue?

[mkzie2]

@getusha Sorry, please correct me if I was wrong, but I cannot see any "Not here" page in this video.

[huult]

@getusha This issue occurred since version 9.0.16-0. I am tracing which PRs in this tag, 9.0.16-0, have caused this issue.

[huult]

@getusha this happend by this PR. This happend by this change.

Steps to test:

1. Check out tag 9.0.15-4 (which does not have this issue).
2. Copy the ReportScreen from tag 9.0.16-0.

You will see the changes, and this issue will occur

•