[HOLD on App#96146][No QA] Show deposit-only settlement accounts for Consolidated Travel Billing

src/languages/en.ts

@@ -6182,6 +6182,7 @@ const translations = {
                     invalidDateRangeError: 'The start date must be before the end date',
                     enabled: 'Consolidated Travel Billing enabled!',
                     enabledDescription: 'All travel spend on this workspace will now be centralized in a monthly bill.',
+                    depositOnly: 'Deposit only',
                 },
                 personalDetailsDescription: 'In order to book travel, please enter your legal name as it appears on your government-issued ID.',
             },

src/libs/CardUtils.ts

@@ -10,6 +10,7 @@ import type {TranslationPaths} from '@src/languages/types';
 import ONYXKEYS from '@src/ONYXKEYS';
 import ROUTES from '@src/ROUTES';
 import type {
+    BankAccount,
     BankAccountList,
     Card,
     CardFeeds,
@@ -512,6 +513,26 @@ function getEligibleBankAccountsForCard(bankAccountsList: OnyxEntry<BankAccountL
     );
 }
 
+/**
+ * Checks whether a verified bank account can only receive deposits (not enabled for debits).
+ */
+function isDepositOnlyBankAccount(bankAccount: BankAccount | undefined): boolean {
+    return !!bankAccount?.accountData?.defaultCredit && !bankAccount?.accountData?.allowDebit && !isBankAccountPartiallySetup(bankAccount?.accountData?.state);
+}
+
+/**
+ * Returns the bank accounts that can settle Consolidated Travel Billing. Includes the standard card-eligible
+ * accounts, plus verified deposit-only accounts when the workspace can settle by invoice instead of an ACH debit.
+ */
+function getEligibleBankAccountsForTravelInvoicing(bankAccountsList: OnyxEntry<BankAccountList>, canUseDepositOnlyAccounts: boolean) {
+    const eligibleBankAccounts = getEligibleBankAccountsForCard(bankAccountsList);
+    if (!canUseDepositOnlyAccounts || !bankAccountsList) {
+        return eligibleBankAccounts;
+    }
+    const depositOnlyBankAccounts = Object.values(bankAccountsList).filter(isDepositOnlyBankAccount);
+    return [...eligibleBankAccounts, ...depositOnlyBankAccounts];
+}
+
 function getConnectionBankAccountsForReconciliation(connections: OnyxEntry<Partial<Connections>>, connectionName: PolicyConnectionName | undefined): Array<{id: string; name: string}> {
     if (!connections || !connectionName) {
         return [];
@@ -2029,6 +2050,8 @@ export {
     getTranslationKeyForCardStatus,
     maskPin,
     getEligibleBankAccountsForCard,
+    getEligibleBankAccountsForTravelInvoicing,
+    isDepositOnlyBankAccount,
     sortCardsByCardholderName,
     isCurrencySupportedForECards,
     getCardFeedIcon,

src/pages/workspace/travel/WorkspaceTravelInvoicingSection.tsx

@@ -27,7 +27,7 @@
     retryTravelCardsProvisioning,
 } from '@libs/actions/TravelInvoicing';
 import {getLastFourDigits} from '@libs/BankAccountUtils';
-import {getCardSettings, getEligibleBankAccountsForCard} from '@libs/CardUtils';
+import {getCardSettings, getEligibleBankAccountsForTravelInvoicing} from '@libs/CardUtils';
 import createDynamicRoute from '@libs/Navigation/helpers/dynamicRoutesUtils/createDynamicRoute';
 import Navigation from '@libs/Navigation/Navigation';
 import {areTravelPersonalDetailsMissing} from '@libs/PersonalDetailsUtils';
@@ -147,7 +147,7 @@
 
     // Bank account eligibility for toggle handler
     const isSetupUnfinished = hasInProgressUSDVBBA(reimbursementAccount?.achData);
-    const eligibleBankAccounts = getEligibleBankAccountsForCard(bankAccountList);
+    const eligibleBankAccounts = getEligibleBankAccountsForTravelInvoicing(bankAccountList, isPayByInvoice);
 
     // Determine if Travel Invoicing is enabled based on isEnabled field
     const isTravelInvoicingEnabled = getIsTravelInvoicingEnabled(travelSettings);

src/pages/workspace/travel/WorkspaceTravelInvoicingSettlementAccountPage.tsx

@@ -14,9 +14,9 @@
 
 import {configureTravelInvoicingForPolicy, setTravelInvoicingSettlementAccount} from '@libs/actions/TravelInvoicing';
 import {getLastFourDigits} from '@libs/BankAccountUtils';
-import {getCardSettings, getEligibleBankAccountsForCard} from '@libs/CardUtils';
+import {getCardSettings, getEligibleBankAccountsForTravelInvoicing, isDepositOnlyBankAccount} from '@libs/CardUtils';
 import type {PlatformStackScreenProps} from '@libs/Navigation/PlatformStackNavigation/types';
-import {getIsTravelInvoicingEnabled, getTravelInvoicingCardSettingsKey} from '@libs/TravelInvoicingUtils';
+import {getIsTravelBillingPayByInvoice, getIsTravelInvoicingEnabled, getTravelInvoicingCardSettingsKey} from '@libs/TravelInvoicingUtils';
 
 import Navigation from '@navigation/Navigation';
 import type {SettingsNavigatorParamList} from '@navigation/types';
@@ -48,7 +48,8 @@
     const isSuccess = !!cardSettings?.isSuccess;
     const isTravelInvoicingEnabled = getIsTravelInvoicingEnabled(travelSettings);
     const paymentBankAccountID = travelSettings?.paymentBankAccountID;
-    const eligibleBankAccounts = getEligibleBankAccountsForCard(bankAccountsList);
+    const canUseDepositOnlyAccounts = getIsTravelBillingPayByInvoice(travelSettings);
+    const eligibleBankAccounts = getEligibleBankAccountsForTravelInvoicing(bankAccountsList, canUseDepositOnlyAccounts);
 
     const getVerificationState = () => {
         if (cardOnWaitlist) {
@@ -69,12 +70,15 @@
         const bankName = (bankAccount.accountData?.addressName ?? '') as BankName;
         const bankAccountNumber = bankAccount.accountData?.accountNumber ?? '';
         const bankAccountID = bankAccount.accountData?.bankAccountID ?? bankAccount.methodID;
+        const accountEndingIn = `${translate('workspace.expensifyCard.accountEndingIn')} ${getLastFourDigits(bankAccountNumber)}`;
 
         return {
             value: bankAccountID,
             text: bankAccount.title,
             leftElement: <BankAccountListItemLeftElement bankName={bankName} />,
-            alternateText: `${translate('workspace.expensifyCard.accountEndingIn')} ${getLastFourDigits(bankAccountNumber)}`,
+            alternateText: isDepositOnlyBankAccount(bankAccount)
+                ? `${accountEndingIn} ${CONST.DOT_SEPARATOR} ${translate('workspace.moreFeatures.travel.travelInvoicing.depositOnly')}`
+                : accountEndingIn,
             keyForList: bankAccountID?.toString() ?? '',
             isSelected: bankAccountID === paymentBankAccountID,
         };

tests/unit/CardUtilsTest.ts

@@ -42,6 +42,7 @@ import {
     getDisplayableThirdPartyCards,
     getDomainByFundID,
     getEligibleBankAccountsForCard,
+    getEligibleBankAccountsForTravelInvoicing,
     getEligibleBankAccountsForUkEuCard,
     getFeedNameForDisplay,
     getFeedType,
@@ -4397,6 +4398,48 @@ describe('getEligibleBankAccountsForCard', () => {
     });
 });
 
+describe('getEligibleBankAccountsForTravelInvoicing', () => {
+    const debitBusinessAccount: BankAccountList = {
+        '1': {
+            accountData: {type: CONST.BANK_ACCOUNT.TYPE.BUSINESS, allowDebit: true, state: CONST.BANK_ACCOUNT.STATE.OPEN},
+            bankCurrency: 'USD',
+            bankCountry: 'US',
+        },
+    };
+
+    const depositOnlyAccount: BankAccountList = {
+        '2': {
+            accountData: {type: CONST.BANK_ACCOUNT.TYPE.BUSINESS, allowDebit: false, defaultCredit: true, state: CONST.BANK_ACCOUNT.STATE.OPEN},
+            bankCurrency: 'USD',
+            bankCountry: 'US',
+        },
+    };
+
+    const partiallySetupDepositOnlyAccount: BankAccountList = {
+        '3': {
+            accountData: {type: CONST.BANK_ACCOUNT.TYPE.BUSINESS, allowDebit: false, defaultCredit: true, state: CONST.BANK_ACCOUNT.STATE.SETUP},
+            bankCurrency: 'USD',
+            bankCountry: 'US',
+        },
+    };
+
+    it('excludes deposit-only accounts when deposit-only accounts are not allowed', () => {
+        const result = getEligibleBankAccountsForTravelInvoicing({...debitBusinessAccount, ...depositOnlyAccount}, false);
+        expect(result).toHaveLength(1);
+        expect(result.at(0)?.accountData?.allowDebit).toBe(true);
+    });
+
+    it('includes verified deposit-only accounts alongside debit-eligible accounts when allowed', () => {
+        const result = getEligibleBankAccountsForTravelInvoicing({...debitBusinessAccount, ...depositOnlyAccount}, true);
+        expect(result).toHaveLength(2);
+    });
+
+    it('always excludes partially set up deposit-only accounts', () => {
+        expect(getEligibleBankAccountsForTravelInvoicing(partiallySetupDepositOnlyAccount, true)).toHaveLength(0);
+        expect(getEligibleBankAccountsForTravelInvoicing(partiallySetupDepositOnlyAccount, false)).toHaveLength(0);
+    });
+});
+
 describe('getEligibleBankAccountsForUkEuCard', () => {
     it('excludes partially set up accounts', () => {
         const bankAccounts: BankAccountList = {