From 7b05dcbd93f0b07f13bf88f0049bb41dfb4bd87f Mon Sep 17 00:00:00 2001 From: Srikar Parsi <48188732+srikarparsi@users.noreply.github.com> Date: Fri, 13 Mar 2026 13:16:59 -0400 Subject: [PATCH] Revert "fix: Confirm page shows Amount field with 0.00 amount when receipt is scanning v2" --- src/components/MoneyRequestConfirmationList.tsx | 8 +------- src/libs/actions/IOU/index.ts | 11 ----------- 2 files changed, 1 insertion(+), 18 deletions(-) diff --git a/src/components/MoneyRequestConfirmationList.tsx b/src/components/MoneyRequestConfirmationList.tsx index 7287cd3038d0f..c5fc3b3280dfc 100644 --- a/src/components/MoneyRequestConfirmationList.tsx +++ b/src/components/MoneyRequestConfirmationList.tsx @@ -55,7 +55,6 @@ import { hasRoute as hasRouteUtil, hasTaxRateWithMatchingValue, isMerchantMissing, - isScanning, isScanRequest as isScanRequestUtil, } from '@libs/TransactionUtils'; import {getIsViolationFixed} from '@libs/Violations/ViolationsUtils'; @@ -405,12 +404,7 @@ function MoneyRequestConfirmationList({ amountToBeUsed = perDiemRequestAmount; } - let formattedAmount = convertToDisplayString(amountToBeUsed, isDistanceRequest ? currency : iouCurrencyCode); - if (isDistanceRequestWithPendingRoute) { - formattedAmount = ''; - } else if (isScanning(transaction)) { - formattedAmount = translate('iou.receiptStatusTitle'); - } + const formattedAmount = isDistanceRequestWithPendingRoute ? '' : convertToDisplayString(amountToBeUsed, isDistanceRequest ? currency : iouCurrencyCode); const formattedAmountPerAttendee = isDistanceRequestWithPendingRoute || isScanRequest ? '' diff --git a/src/libs/actions/IOU/index.ts b/src/libs/actions/IOU/index.ts index f6623093c4ad7..67d2d8267594e 100644 --- a/src/libs/actions/IOU/index.ts +++ b/src/libs/actions/IOU/index.ts @@ -1430,21 +1430,12 @@ function startDistanceRequest( } } -function setMoneyRequestReceiptState(transactionID: string, isDraft: boolean) { - if (!isDraft) { - return; - } - Onyx.merge(`${ONYXKEYS.COLLECTION.TRANSACTION_DRAFT}${transactionID}`, {receipt: {state: CONST.IOU.RECEIPT_STATE.OPEN}}); -} - function setMoneyRequestAmount(transactionID: string, amount: number, currency: string, shouldShowOriginalAmount = false) { Onyx.merge(`${ONYXKEYS.COLLECTION.TRANSACTION_DRAFT}${transactionID}`, {amount, currency, shouldShowOriginalAmount}); - setMoneyRequestReceiptState(transactionID, true); } function setMoneyRequestCreated(transactionID: string, created: string, isDraft: boolean) { Onyx.merge(`${isDraft ? ONYXKEYS.COLLECTION.TRANSACTION_DRAFT : ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`, {created}); - setMoneyRequestReceiptState(transactionID, isDraft); } function setMoneyRequestDateAttribute(transactionID: string, start: string, end: string) { @@ -1458,12 +1449,10 @@ function setMoneyRequestCurrency(transactionID: string, currency: string, isEdit function setMoneyRequestDescription(transactionID: string, comment: string, isDraft: boolean) { Onyx.merge(`${isDraft ? ONYXKEYS.COLLECTION.TRANSACTION_DRAFT : ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`, {comment: {comment: comment.trim()}}); - setMoneyRequestReceiptState(transactionID, isDraft); } function setMoneyRequestMerchant(transactionID: string, merchant: string, isDraft: boolean) { Onyx.merge(`${isDraft ? ONYXKEYS.COLLECTION.TRANSACTION_DRAFT : ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`, {merchant}); - setMoneyRequestReceiptState(transactionID, isDraft); } function setMoneyRequestAttendees(transactionID: string, attendees: Attendee[], isDraft: boolean) {