Skip to content

Commit 51b97ec

Browse files
Merge pull request #57785 from Expensify/cristi_worksace-address-when-enabling-travel
Don't ask for workspace address if public domain user can't enable Travel
2 parents 8ff59e5 + f6240c7 commit 51b97ec

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

src/components/BookTravelButton.tsx

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,6 @@ function BookTravelButton({text}: BookTravelButtonProps) {
5858
return;
5959
}
6060

61-
// Spotnana requires an address anytime an entity is created for a policy
62-
if (isEmptyObject(policy?.address)) {
63-
Navigation.navigate(ROUTES.WORKSPACE_OVERVIEW_ADDRESS.getRoute(policy?.id, Navigation.getActiveRoute()));
64-
return;
65-
}
66-
6761
const isPolicyProvisioned = policy?.travelSettings?.spotnanaCompanyID ?? policy?.travelSettings?.associatedTravelDomainAccountID;
6862
if (policy?.travelSettings?.hasAcceptedTerms ?? (travelSettings?.hasAcceptedTerms && isPolicyProvisioned)) {
6963
openTravelDotLink(policy?.id)
@@ -92,6 +86,9 @@ function BookTravelButton({text}: BookTravelButtonProps) {
9286
const adminDomains = getAdminsPrivateEmailDomains(policy);
9387
if (adminDomains.length === 0) {
9488
Navigation.navigate(ROUTES.TRAVEL_PUBLIC_DOMAIN_ERROR);
89+
} else if (isEmptyObject(policy?.address)) {
90+
// Spotnana requires an address anytime an entity is created for a policy
91+
Navigation.navigate(ROUTES.WORKSPACE_OVERVIEW_ADDRESS.getRoute(policy?.id, Navigation.getActiveRoute()));
9592
} else if (adminDomains.length === 1) {
9693
navigateToAcceptTerms(adminDomains.at(0) ?? CONST.TRAVEL.DEFAULT_DOMAIN);
9794
} else {

0 commit comments

Comments
 (0)