Skip to content

Commit

Permalink
Merge pull request #29388 from Expensify/francois-fix-money-request-m…
Browse files Browse the repository at this point in the history
…erchant

Don't show duplicate merchant input for distance requests
  • Loading branch information
roryabraham authored Oct 12, 2023
2 parents 828b8b3 + 93c89d6 commit c5d6f89
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/components/MoneyRequestConfirmationList.js
Original file line number Diff line number Diff line change
Expand Up @@ -211,9 +211,9 @@ function MoneyRequestConfirmationList(props) {
// Do not hide fields in case of send money request
const shouldShowAllFields = props.isDistanceRequest || shouldExpandFields || !props.shouldShowSmartScanFields || isTypeSend || props.isEditingSplitBill;

// In Send Money flow, we don't allow the Merchant or Date to be edited.
// In Send Money flow, we don't allow the Merchant or Date to be edited. For distance requests, don't show the merchant as there's already another "Distance" menu item
const shouldShowDate = shouldShowAllFields && !isTypeSend;
const shouldShowMerchant = shouldShowAllFields && !isTypeSend;
const shouldShowMerchant = shouldShowAllFields && !isTypeSend && !props.isDistanceRequest;

// Fetches the first tag list of the policy
const policyTag = PolicyUtils.getTag(props.policyTags);
Expand Down

0 comments on commit c5d6f89

Please sign in to comment.