Skip to content

Commit 6ea15ed

Browse files
authored
Merge pull request #44508 from etCoderDysto/fixHold
fix: Hold option appears for invoice
2 parents f36ebf5 + fd9d28b commit 6ea15ed

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/components/MoneyReportHeader.tsx

+3-2
Original file line numberDiff line numberDiff line change
@@ -248,14 +248,15 @@ function MoneyReportHeader({policy, report: moneyRequestReport, transactionThrea
248248
const isHoldCreator = ReportUtils.isHoldCreator(transaction, moneyRequestReport?.reportID) && isRequestIOU;
249249
const isTrackExpenseReport = ReportUtils.isTrackExpenseReport(moneyRequestReport);
250250
const canModifyStatus = !isTrackExpenseReport && (isPolicyAdmin || isActionOwner || isApprover);
251-
if (isOnHold && (isHoldCreator || (!isRequestIOU && canModifyStatus))) {
251+
const isInvoiceReport = ReportUtils.isInvoiceReport(moneyRequestReport);
252+
if (isOnHold && (isHoldCreator || (!isRequestIOU && canModifyStatus)) && !isInvoiceReport) {
252253
threeDotsMenuItems.push({
253254
icon: Expensicons.Stopwatch,
254255
text: translate('iou.unholdExpense'),
255256
onSelected: () => changeMoneyRequestStatus(),
256257
});
257258
}
258-
if (!isOnHold && (isRequestIOU || canModifyStatus) && !isScanning) {
259+
if (!isOnHold && (isRequestIOU || canModifyStatus) && !isScanning && !isInvoiceReport) {
259260
threeDotsMenuItems.push({
260261
icon: Expensicons.Stopwatch,
261262
text: translate('iou.hold'),

0 commit comments

Comments
 (0)