Merge pull request #44508 from etCoderDysto/fixHold

fix: Hold option appears for invoice
Expensify · Jun 26, 2024 · 6ea15ed · 6ea15ed
2 parents f36ebf5 + fd9d28b
commit 6ea15ed
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/components/MoneyReportHeader.tsx b/src/components/MoneyReportHeader.tsx
@@ -248,14 +248,15 @@ function MoneyReportHeader({policy, report: moneyRequestReport, transactionThrea
         const isHoldCreator = ReportUtils.isHoldCreator(transaction, moneyRequestReport?.reportID) && isRequestIOU;
         const isTrackExpenseReport = ReportUtils.isTrackExpenseReport(moneyRequestReport);
         const canModifyStatus = !isTrackExpenseReport && (isPolicyAdmin || isActionOwner || isApprover);
-        if (isOnHold && (isHoldCreator || (!isRequestIOU && canModifyStatus))) {
+        const isInvoiceReport = ReportUtils.isInvoiceReport(moneyRequestReport);
+        if (isOnHold && (isHoldCreator || (!isRequestIOU && canModifyStatus)) && !isInvoiceReport) {
             threeDotsMenuItems.push({
                 icon: Expensicons.Stopwatch,
                 text: translate('iou.unholdExpense'),
                 onSelected: () => changeMoneyRequestStatus(),
             });
         }
-        if (!isOnHold && (isRequestIOU || canModifyStatus) && !isScanning) {
+        if (!isOnHold && (isRequestIOU || canModifyStatus) && !isScanning && !isInvoiceReport) {
             threeDotsMenuItems.push({
                 icon: Expensicons.Stopwatch,
                 text: translate('iou.hold'),