[HOLD for payment 2024-01-26] [HOLD for payment 2024-01-25] Cancel payment - IOU is still marked as paid when payment is canceled unless report is revisited

[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.16-3
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:

Issue found when executing PR #32674

Action Performed:

1. [User A] Request money from User B
2. [User B] Click on IOU preview > Pay elsewhere
3. [User B] Click 3-dot menu > Cancel payment > Cancel payment

Expected Result:

On User A's end, the IOU preview will no longer be marked as paid when the payment is cancelled

Actual Result:

On User A's end, the IOU preview is still marked as paid unless the IOU report is revisited

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

Bug6324149_1703264039124.bandicam_2023-12-22_17-31-55-857__1_.mp4

View all open jobs on GitHub

[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 @johnmlee101 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[paultsimura]

Proposal

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

When canceling the payment, its status is not immediately changed to "canceled".

What is the root cause of that problem?

This is a combined BE and FE issue. This is also seen on the Admin's side: the "canceled" label is not added at all.
We do not update the iouReport.isCancelledIOU: true optimistically when canceling the payment.

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

When canceling the payment, we should optimistically set isCancelledIOU: true here. This will make the changes immediate for the Admin and ensure the Offline-first approach:

App/src/libs/actions/IOU.js

Lines 3114 to 3123 in b0e038f

	onyxMethod: Onyx.METHOD.MERGE,
	key: `${ONYXKEYS.COLLECTION.REPORT}${expenseReport.reportID}`,
	value: {
	...expenseReport,
	lastMessageText: lodashGet(optimisticReportAction, 'message.0.text', ''),
	lastMessageHtml: lodashGet(optimisticReportAction, 'message.0.html', ''),
	state: isFree ? CONST.REPORT.STATE.SUBMITTED : CONST.REPORT.STATE.OPEN,
	stateNum: isFree ? CONST.REPORT.STATE_NUM.PROCESSING : CONST.REPORT.STATE.OPEN,
	statusNum: isFree ? CONST.REPORT.STATUS.SUBMITTED : CONST.REPORT.STATE.OPEN,
	},

Also, we should update the BE: Pusher should update the IOU Report on the member side when it's marked as isCancelledIOU: true via the CancelPayment API call.

What alternative solutions did you explore? (Optional)