IOU - LHN for IOU report still shows the requested amount when IOU is deleted

[kbecciv]

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: v1.4.44-0
Reproducible in staging?: y
Reproducible in production?: y
Issue reported by: Applause - Internal Team
Issue found when executing PR: #36557

Action Performed:

1. Go to staging.new.expensify.com
2. Go to 1:1 DM.
3. Create a manual request.
4. Go to request details page.
5. Send a message.
6. Click 3-dot menu > Delete request.
7. Return to IOU report.

Expected Result:

LHN for IOU report will show [Deleted request].

Actual Result:

LHN for IOU report still shows the requested amount when IOU is deleted.

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

Bug6394391_1709056655937.bandicam_2024-02-27_06-57-33-774.mp4

View all open jobs on GitHub

[melvin-bot]

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

[kbecciv]

@stephanieelliott 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

[kbecciv]

We think that this bug might be related to #vip-bills
CC @davidcardoza

[dukenv0307]

Proposal

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

• IOU - LHN for IOU report still shows the requested amount when IOU is deleted

What is the root cause of that problem?

• In here we do not check if the action is deleted or not, so that leads to the bug.

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

• We need to add check if the action is deleted in that case. If true, display [Deleted request] instead. Do it by adding to here:

        if(ReportActionsUtils.isDeletedAction(reportAction) && ReportActionsUtils.isTransactionThread(reportAction)){
            return Localize.translateLocal('parentReportAction.deletedRequest')
        }

or use isDeletedParentAction rather than isDeletedAction

• Also, we need to consider the case isReversedTransaction(reportAction): true

What alternative solutions did you explore? (Optional)

• NA

[melvin-bot]

@stephanieelliott Eep! 4 days overdue now. Issues have feelings too...

[stephanieelliott]

I don't really think this is a bug, as long as the X owes Y 0.00 stays current, it's not unexpected that the LHN shows the most recent request made (even if it was deleted afterward). I'm not sure if it was a conscious UI decision or something that wasn't given consideration, but I don't think we need to prioritize updating this behavior.