From 8d215d01fa7e61892382d74713602e77625b6261 Mon Sep 17 00:00:00 2001 From: "Daniel Gale-Rosen (via MelvinBot)" Date: Tue, 5 May 2026 18:22:28 +0000 Subject: [PATCH] Add showInReview to merged prohibited expense violations The mergeProhibitedViolations function creates a merged violation object without showInReview: true, so the transaction preview filters it out when checking for violations to display. This adds the flag to align with the established pattern used by all other violation creators. Co-authored-by: Daniel Gale-Rosen --- src/libs/TransactionUtils/index.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/src/libs/TransactionUtils/index.ts b/src/libs/TransactionUtils/index.ts index 29845965dbfd..0f149303592d 100644 --- a/src/libs/TransactionUtils/index.ts +++ b/src/libs/TransactionUtils/index.ts @@ -1673,6 +1673,7 @@ function mergeProhibitedViolations(transactionViolations: TransactionViolations) prohibitedExpenseRule: prohibitedExpenses, }, type: CONST.VIOLATION_TYPES.VIOLATION, + showInReview: true, }; return [...transactionViolations.filter((violation: TransactionViolation) => violation.name !== CONST.VIOLATIONS.PROHIBITED_EXPENSE), mergedProhibitedViolations];