Skip to content

Commit 3eca10a

Browse files
mountinymountiny
authored
Merge pull request #58518 from Expensify/revert-57641-QBOAutoSync
[CP Staging] Revert "New Feature: "When to export" selector for auto-sync for Quickbooks Online"
2 parents 6d00a6b + 6c06582 commit 3eca10a

File tree

21 files changed

+82
-333
lines changed

21 files changed

+82
-333
lines changed

‎src/CONST.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1943,7 +1943,6 @@ const CONST = {
19431943
AUTO_CREATE_VENDOR: 'autoCreateVendor',
19441944
REIMBURSEMENT_ACCOUNT_ID: 'reimbursementAccountID',
19451945
COLLECTION_ACCOUNT_ID: 'collectionAccountID',
1946-
ACCOUNTING_METHOD: 'accountingMethod',
19471946
},
19481947

19491948
XERO_CONFIG: {

‎src/ROUTES.ts

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1176,19 +1176,11 @@ const ROUTES = {
11761176
},
11771177
WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_ACCOUNT_SELECTOR: {
11781178
route: 'settings/workspaces/:policyID/accounting/quickbooks-online/account-selector',
1179-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/accounting/quickbooks-online/account-selector` as const,
1179+
getRoute: (policyID: string) => `settings/workspaces/${policyID}/accounting/quickbooks-online/account-selector` as const,
11801180
},
11811181
WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_INVOICE_ACCOUNT_SELECTOR: {
11821182
route: 'settings/workspaces/:policyID/accounting/quickbooks-online/invoice-account-selector',
1183-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/accounting/quickbooks-online/invoice-account-selector` as const,
1184-
},
1185-
WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_AUTO_SYNC: {
1186-
route: 'settings/workspaces/:policyID/connections/quickbooks-online/advanced/autosync',
1187-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/connections/quickbooks-online/advanced/autosync` as const,
1188-
},
1189-
WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_ACCOUNTING_METHOD: {
1190-
route: 'settings/workspaces/:policyID/connections/quickbooks-online/advanced/autosync/accounting-method',
1191-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/connections/quickbooks-online/advanced/autosync/accounting-method` as const,
1183+
getRoute: (policyID: string) => `settings/workspaces/${policyID}/accounting/quickbooks-online/invoice-account-selector` as const,
11921184
},
11931185
WORKSPACE_ACCOUNTING_CARD_RECONCILIATION: {
11941186
route: 'settings/workspaces/:policyID/accounting/:connection/card-reconciliation',
@@ -2096,11 +2088,11 @@ const ROUTES = {
20962088
},
20972089
POLICY_ACCOUNTING_NETSUITE_AUTO_SYNC: {
20982090
route: 'settings/workspaces/:policyID/connections/netsuite/advanced/autosync',
2099-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/connections/netsuite/advanced/autosync` as const,
2091+
getRoute: (policyID: string) => `settings/workspaces/${policyID}/connections/netsuite/advanced/autosync` as const,
21002092
},
21012093
POLICY_ACCOUNTING_NETSUITE_ACCOUNTING_METHOD: {
21022094
route: 'settings/workspaces/:policyID/connections/netsuite/advanced/autosync/accounting-method',
2103-
getRoute: (policyID: string | undefined) => `settings/workspaces/${policyID}/connections/netsuite/advanced/autosync/accounting-method` as const,
2095+
getRoute: (policyID: string) => `settings/workspaces/${policyID}/connections/netsuite/advanced/autosync/accounting-method` as const,
21042096
},
21052097
POLICY_ACCOUNTING_NSQS_SETUP: {
21062098
route: 'settings/workspaces/:policyID/accounting/nsqs/setup',

‎src/SCREENS.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -371,8 +371,6 @@ const SCREENS = {
371371
QUICKBOOKS_ONLINE_CLASSES_DISPLAYED_AS: 'Policy_Accounting_Quickbooks_Online_Import_Classes_Displayed_As',
372372
QUICKBOOKS_ONLINE_CUSTOMERS_DISPLAYED_AS: 'Policy_Accounting_Quickbooks_Online_Import_Customers_Displayed_As',
373373
QUICKBOOKS_ONLINE_LOCATIONS_DISPLAYED_AS: 'Policy_Accounting_Quickbooks_Online_Import_Locations_Displayed_As',
374-
QUICKBOOKS_ONLINE_AUTO_SYNC: 'Policy_Accounting_Quickbooks_Online_Auto_Sync',
375-
QUICKBOOKS_ONLINE_ACCOUNTING_METHOD: 'Policy_Accounting_Quickbooks_Online_Accounting_Method',
376374
QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_SELECT: 'Workspace_Accounting_Quickbooks_Desktop_Export_Company_Card_Expense_Account_Select',
377375
QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_COMPANY_CARD_SELECT: 'Workspace_Accounting_Quickbooks_Desktop_Export_Company_Card_Expense_Select',
378376
QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT: 'Workspace_Accounting_Quickbooks_Desktop_Export_Company_Card_Expense',

‎src/languages/en.ts

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2918,7 +2918,6 @@ const translations = {
29182918
locations: 'Locations',
29192919
customers: 'Customers/projects',
29202920
accountsDescription: 'Your QuickBooks Online chart of accounts will import into Expensify as categories.',
2921-
autoSyncDescription: 'Sync QuickBooks Online and Expensify automatically, every day. Export finalized report in realtime',
29222921
accountsSwitchTitle: 'Choose to import new accounts as enabled or disabled categories.',
29232922
accountsSwitchDescription: 'Enabled categories will be available for members to select when creating their expenses.',
29242923
classesDescription: 'Choose how to handle QuickBooks Online classes in Expensify.',
@@ -3253,6 +3252,18 @@ const translations = {
32533252
[CONST.NETSUITE_REPORTS_APPROVAL_LEVEL.REPORTS_APPROVED_BOTH]: 'Supervisor and accounting approved',
32543253
},
32553254
},
3255+
accountingMethods: {
3256+
label: 'When to Export',
3257+
description: 'Choose when to export the expenses:',
3258+
values: {
3259+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Accrual',
3260+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Cash',
3261+
},
3262+
alternateText: {
3263+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Out-of-pocket expenses will export when final approved',
3264+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Out-of-pocket expenses will export when paid',
3265+
},
3266+
},
32563267
exportVendorBillsTo: {
32573268
label: 'Vendor bill approval level',
32583269
description: 'Once a vendor bill is approved in Expensify and exported to NetSuite, you can set an additional level of approval in NetSuite prior to posting.',
@@ -3546,18 +3557,6 @@ const translations = {
35463557
}
35473558
},
35483559
},
3549-
accountingMethods: {
3550-
label: 'When to Export',
3551-
description: 'Choose when to export the expenses:',
3552-
values: {
3553-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Accrual',
3554-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Cash',
3555-
},
3556-
alternateText: {
3557-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Out-of-pocket expenses will export when final approved',
3558-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Out-of-pocket expenses will export when paid',
3559-
},
3560-
},
35613560
multiConnectionSelector: {
35623561
title: ({connectionName}: ConnectionNameParams) => `${CONST.POLICY.CONNECTIONS.NAME_USER_FRIENDLY[connectionName]} setup`,
35633562
description: ({connectionName}: ConnectionNameParams) => `Select your ${CONST.POLICY.CONNECTIONS.NAME_USER_FRIENDLY[connectionName]} version to continue.`,

‎src/languages/es.ts

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2944,7 +2944,6 @@ const translations = {
29442944
locations: 'Lugares',
29452945
customers: 'Clientes/proyectos',
29462946
accountsDescription: 'Tu plan de cuentas de QuickBooks Online se importará a Expensify como categorías.',
2947-
autoSyncDescription: 'Sincroniza QuickBooks Online y Expensify automáticamente, todos los días. Exporta el informe finalizado en tiempo real',
29482947
accountsSwitchTitle: 'Elige importar cuentas nuevas como categorías activadas o desactivadas.',
29492948
accountsSwitchDescription: 'Las categorías activas estarán disponibles para ser escogidas cuando se crea un gasto.',
29502949
classesDescription: 'Elige cómo gestionar las clases de QuickBooks Online en Expensify.',
@@ -3291,6 +3290,18 @@ const translations = {
32913290
[CONST.NETSUITE_REPORTS_APPROVAL_LEVEL.REPORTS_APPROVED_BOTH]: 'Aprobado por supervisor y contabilidad',
32923291
},
32933292
},
3293+
accountingMethods: {
3294+
label: 'Cuándo Exportar',
3295+
description: 'Elige cuándo exportar los gastos:',
3296+
values: {
3297+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Devengo',
3298+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Efectivo',
3299+
},
3300+
alternateText: {
3301+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Los gastos por cuenta propia se exportarán cuando estén aprobados definitivamente',
3302+
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Los gastos por cuenta propia se exportarán cuando estén pagados',
3303+
},
3304+
},
32943305
exportVendorBillsTo: {
32953306
label: 'Nivel de aprobación de facturas de proveedores',
32963307
description:
@@ -3585,18 +3596,6 @@ const translations = {
35853596
}
35863597
},
35873598
},
3588-
accountingMethods: {
3589-
label: 'Cuándo Exportar',
3590-
description: 'Elige cuándo exportar los gastos:',
3591-
values: {
3592-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Devengo',
3593-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Efectivo',
3594-
},
3595-
alternateText: {
3596-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.ACCRUAL]: 'Los gastos por cuenta propia se exportarán cuando estén aprobados definitivamente',
3597-
[COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD.CASH]: 'Los gastos por cuenta propia se exportarán cuando estén pagados',
3598-
},
3599-
},
36003599
multiConnectionSelector: {
36013600
title: ({connectionName}: ConnectionNameParams) => `${CONST.POLICY.CONNECTIONS.NAME_USER_FRIENDLY[connectionName]} configuración`,
36023601
description: ({connectionName}: ConnectionNameParams) => `Selecciona tu versión de ${CONST.POLICY.CONNECTIONS.NAME_USER_FRIENDLY[connectionName]} para continuar.`,

‎src/libs/API/parameters/UpdateQuickbooksOnlineAccountingMethodParams.ts

Lines changed: 0 additions & 9 deletions
This file was deleted.

‎src/libs/API/parameters/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ export type {default as SyncPolicyToQuickbooksOnlineParams} from './SyncPolicyTo
2020
export type {default as SyncPolicyToXeroParams} from './SyncPolicyToXeroParams';
2121
export type {default as SyncPolicyToNetSuiteParams} from './SyncPolicyToNetSuiteParams';
2222
export type {default as UpdateNetSuiteAccountingMethodParams} from './UpdateNetSuiteAccountingMethodParams';
23-
export type {default as UpdateQuickbooksOnlineAccountingMethodParams} from './UpdateQuickbooksOnlineAccountingMethodParams';
2423
export type {default as SyncPolicyToQuickbooksDesktopParams} from './SyncPolicyToQuickbooksDesktopParams';
2524
export type {default as DeleteContactMethodParams} from './DeleteContactMethodParams';
2625
export type {default as DeletePaymentBankAccountParams} from './DeletePaymentBankAccountParams';

‎src/libs/API/types.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,6 @@ const WRITE_COMMANDS = {
268268
UPDATE_QUICKBOOKS_ONLINE_SYNC_PEOPLE: 'UpdateQuickbooksOnlineSyncPeople',
269269
UPDATE_QUICKBOOKS_ONLINE_REIMBURSEMENT_ACCOUNT_ID: 'UpdateQuickbooksOnlineReimbursementAccountID',
270270
UPDATE_QUICKBOOKS_ONLINE_EXPORT: 'UpdateQuickbooksOnlineExport',
271-
UPDATE_QUICKBOOKS_ONLINE_ACCOUNTING_METHOD: 'UpdateQuickbooksOnlineAccountingMethod',
272271
UPDATE_QUICKBOOKS_DESKTOP_EXPORT_DATE: 'UpdateQuickbooksDesktopExportDate',
273272
UPDATE_MANY_POLICY_CONNECTION_CONFIGS: 'UpdateManyPolicyConnectionConfigurations',
274273
UPDATE_QUICKBOOKS_DESKTOP_NON_REIMBURSABLE_EXPENSES_EXPORT_DESTINATION: 'UpdateQuickbooksDesktopNonReimbursableExpensesExportDestination',
@@ -751,7 +750,6 @@ type WriteCommandParameters = {
751750
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_ONLINE_EXPORT_DATE]: Parameters.UpdateQuickbooksOnlineGenericTypeParams;
752751
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_ONLINE_NON_REIMBURSABLE_EXPENSES_ACCOUNT]: Parameters.UpdateQuickbooksOnlineGenericTypeParams;
753752
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_ONLINE_COLLECTION_ACCOUNT_ID]: Parameters.UpdateQuickbooksOnlineGenericTypeParams;
754-
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_ONLINE_ACCOUNTING_METHOD]: Parameters.UpdateQuickbooksOnlineAccountingMethodParams;
755753
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_DESKTOP_EXPORT_DATE]: Parameters.UpdateQuickbooksDesktopGenericTypeParams;
756754
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_DESKTOP_MARK_CHECKS_TO_BE_PRINTED]: Parameters.UpdateQuickbooksDesktopGenericTypeParams;
757755
[WRITE_COMMANDS.UPDATE_QUICKBOOKS_DESKTOP_AUTO_CREATE_VENDOR]: Parameters.UpdateQuickbooksDesktopGenericTypeParams;

‎src/libs/Navigation/AppNavigator/ModalStackNavigators/index.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -353,9 +353,6 @@ const SettingsModalStackNavigator = createModalStackNavigator<SettingsNavigatorP
353353
require<ReactComponentModule>('@pages/workspace/accounting/qbo/export/QuickbooksCompanyCardExpenseAccountSelectPage').default,
354354
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_COMPANY_CARD_EXPENSE_ACCOUNT_COMPANY_CARD_SELECT]: () =>
355355
require<ReactComponentModule>('../../../../pages/workspace/accounting/qbo/export/QuickbooksCompanyCardExpenseAccountSelectCardPage').default,
356-
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_AUTO_SYNC]: () => require<ReactComponentModule>('../../../../pages/workspace/accounting/qbo/advanced/QuickbooksAutoSyncPage').default,
357-
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_ACCOUNTING_METHOD]: () =>
358-
require<ReactComponentModule>('../../../../pages/workspace/accounting/qbo/advanced/QuickbooksAccountingMethodPage').default,
359356
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_COMPANY_CARD_EXPENSE_ACCOUNT]: () =>
360357
require<ReactComponentModule>('../../../../pages/workspace/accounting/qbo/export/QuickbooksCompanyCardExpenseAccountPage').default,
361358
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_EXPORT_PREFERRED_EXPORTER]: () =>

‎src/libs/Navigation/linkingConfig/RELATIONS/WORKSPACE_TO_RHP.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,6 @@ const WORKSPACE_TO_RHP: Partial<Record<keyof WorkspaceSplitNavigatorParamList, s
5555
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_CLASSES_DISPLAYED_AS,
5656
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_CUSTOMERS_DISPLAYED_AS,
5757
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_LOCATIONS_DISPLAYED_AS,
58-
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_AUTO_SYNC,
59-
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_ACCOUNTING_METHOD,
6058
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_SELECT,
6159
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_COMPANY_CARD_SELECT,
6260
SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT,

‎src/libs/Navigation/linkingConfig/config.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -358,12 +358,6 @@ const config: LinkingOptions<RootNavigatorParamList>['config'] = {
358358
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_LOCATIONS_DISPLAYED_AS]: {
359359
path: ROUTES.POLICY_ACCOUNTING_QUICKBOOKS_ONLINE_LOCATIONS_DISPLAYED_AS.route,
360360
},
361-
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_AUTO_SYNC]: {
362-
path: ROUTES.WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_AUTO_SYNC.route,
363-
},
364-
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_ONLINE_ACCOUNTING_METHOD]: {
365-
path: ROUTES.WORKSPACE_ACCOUNTING_QUICKBOOKS_ONLINE_ACCOUNTING_METHOD.route,
366-
},
367361
[SCREENS.WORKSPACE.ACCOUNTING.QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_SELECT]: {
368362
path: ROUTES.POLICY_ACCOUNTING_QUICKBOOKS_DESKTOP_COMPANY_CARD_EXPENSE_ACCOUNT_SELECT.route,
369363
},

‎src/libs/actions/Policy/Policy.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -723,14 +723,10 @@ function clearSageIntacctErrorField(policyID: string | undefined, fieldName: str
723723
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {connections: {intacct: {config: {errorFields: {[fieldName]: null}}}}});
724724
}
725725

726-
function clearNetSuiteAutoSyncErrorField(policyID: string | undefined) {
726+
function clearNetSuiteAutoSyncErrorField(policyID: string) {
727727
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {connections: {netsuite: {config: {errorFields: {autoSync: null}}}}});
728728
}
729729

730-
function clearQuickbooksOnlineAutoSyncErrorField(policyID: string | undefined) {
731-
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {connections: {quickbooksOnline: {config: {errorFields: {autoSync: null}}}}});
732-
}
733-
734730
function clearNSQSErrorField(policyID: string, fieldName: string) {
735731
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {connections: {netsuiteQuickStart: {config: {errorFields: {[fieldName]: null}}}}});
736732
}
@@ -5035,7 +5031,6 @@ export {
50355031
clearNetSuiteErrorField,
50365032
clearNetSuitePendingField,
50375033
clearNetSuiteAutoSyncErrorField,
5038-
clearQuickbooksOnlineAutoSyncErrorField,
50395034
removeNetSuiteCustomFieldByIndex,
50405035
clearWorkspaceReimbursementErrors,
50415036
setWorkspaceCurrencyDefault,

‎src/libs/actions/connections/NetSuiteCommands.ts

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -766,10 +766,7 @@ function updateNetSuiteExportToNextOpenPeriod(policyID: string, value: boolean,
766766
API.write(WRITE_COMMANDS.UPDATE_NETSUITE_EXPORT_TO_NEXT_OPEN_PERIOD, parameters, onyxData);
767767
}
768768

769-
function updateNetSuiteAutoSync(policyID: string | undefined, value: boolean) {
770-
if (!policyID) {
771-
return;
772-
}
769+
function updateNetSuiteAutoSync(policyID: string, value: boolean) {
773770
const optimisticData: OnyxUpdate[] = [
774771
{
775772
onyxMethod: Onyx.METHOD.MERGE,
@@ -940,13 +937,10 @@ function updateNetSuiteExportReportsTo(
940937
}
941938

942939
function updateNetSuiteAccountingMethod(
943-
policyID: string | undefined,
940+
policyID: string,
944941
accountingMethod: ValueOf<typeof COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD>,
945942
oldAccountingMethod: ValueOf<typeof COMMON_CONST.INTEGRATIONS.ACCOUNTING_METHOD>,
946943
) {
947-
if (!policyID) {
948-
return;
949-
}
950944
const onyxData = updateNetSuiteOnyxData(policyID, CONST.NETSUITE_CONFIG.ACCOUNTING_METHOD, accountingMethod, oldAccountingMethod);
951945

952946
const parameters = {

0 commit comments

Comments
 (0)