Skip to content

Commit

Permalink
Merge pull request #57249 from Expensify/rodrigo-filter-expensifydoma…
Browse files Browse the repository at this point in the history
…ins-travel

Fix domain list when picking domain for travel
  • Loading branch information
MariaHCD authored Feb 24, 2025
2 parents c87ab01 + 2f28a5b commit eb89fbc
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion src/libs/PolicyUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1264,9 +1264,17 @@ function getAdminsPrivateEmailDomains(policy?: Policy) {
domains.push(Str.extractEmailDomain(email).toLowerCase());
return domains;
}, [] as string[]);

const ownerDomains = policy.owner ? [Str.extractEmailDomain(policy.owner).toLowerCase()] : [];

return [...new Set(adminDomains.concat(ownerDomains))].filter((domain) => !isPublicDomain(domain));
const privateDomains = [...new Set(adminDomains.concat(ownerDomains))].filter((domain) => !isPublicDomain(domain));

// If the policy is not owned by Expensify there is no point in showing the domain for provisioning.
if (!isExpensifyTeam(policy.owner)) {
return privateDomains.filter((domain) => domain !== CONST.EXPENSIFY_PARTNER_NAME && domain !== CONST.EMAIL.GUIDES_DOMAIN);
}

return privateDomains;
}

/**
Expand Down

0 comments on commit eb89fbc

Please sign in to comment.