Skip to content

Commit

Permalink
Merge pull request #45148 from bernhardoj/fix/44892-dont-show-unhold-…
Browse files Browse the repository at this point in the history
…for-duplicate-expense

Don't show unhold for expense that is hold because of duplicate
  • Loading branch information
flodnv authored Jul 11, 2024
2 parents b9b0ce2 + 207cd1a commit a8bbd47
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/libs/ReportUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2830,7 +2830,9 @@ function canHoldUnholdReportAction(reportAction: OnyxInputOrEntry<ReportAction>)

const canHoldOrUnholdRequest = !isRequestSettled && !isApproved && !isDeletedParentAction;
const canHoldRequest = canHoldOrUnholdRequest && !isOnHold && (isRequestHoldCreator || (!isRequestIOU && canModifyStatus)) && !isScanning && !!transaction?.reimbursable;
const canUnholdRequest = !!(canHoldOrUnholdRequest && isOnHold && (isRequestHoldCreator || (!isRequestIOU && canModifyStatus))) && !!transaction?.reimbursable;
const canUnholdRequest =
!!(canHoldOrUnholdRequest && isOnHold && !TransactionUtils.isDuplicate(transaction.transactionID, true) && (isRequestHoldCreator || (!isRequestIOU && canModifyStatus))) &&
!!transaction?.reimbursable;

return {canHoldRequest, canUnholdRequest};
}
Expand Down

0 comments on commit a8bbd47

Please sign in to comment.