Skip to content

Commit

Permalink
Merge pull request Expensify#43446 from Expensify/chirag-violation-fix
Browse files Browse the repository at this point in the history
[CP-stag] Removing check for shouldShowHoldMessage
  • Loading branch information
luacmartins authored Jun 11, 2024
2 parents fd91056 + ce6c112 commit 9e4e501
Showing 1 changed file with 1 addition and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -163,11 +163,7 @@ function MoneyRequestPreviewContent({
const isTooLong = violationsCount > 1 || violationMessage.length > 15;
const hasViolationsAndFieldErrors = violationsCount > 0 && hasFieldErrors;

message += ` ${CONST.DOT_SEPARATOR} ${isTooLong || hasViolationsAndFieldErrors ? translate('violations.reviewRequired') : violationMessage}`;
if (shouldShowHoldMessage) {
message += ` ${CONST.DOT_SEPARATOR} ${translate('iou.hold')}`;
}
return message;
return `${message} ${CONST.DOT_SEPARATOR} ${isTooLong || hasViolationsAndFieldErrors ? translate('violations.reviewRequired') : violationMessage}`;
}

const isMerchantMissing = TransactionUtils.isMerchantMissing(transaction);
Expand Down

0 comments on commit 9e4e501

Please sign in to comment.