address comments

src/pages/workspace/accounting/netsuite/advanced/NetSuiteAccountingMethodPage.tsx

@@ -23,14 +23,14 @@ function NetSuiteAccountingMethodPage({policy}: WithPolicyConnectionsProps) {
     const {translate} = useLocalize();
     const policyID = policy?.id ?? '-1';
     const styles = useThemeStyles();
-    const config = policy?.connections?.netsuite.options.config;
-    const accountingMehtod = config?.accountingMethod ?? CONST.NETSUITE_ACCOUNTING_METHODS.CASH;
-    const data: MenuListItem[] = Object.values(CONST.NETSUITE_ACCOUNTING_METHODS).map((accountingMehtodType) => ({
-        value: accountingMehtodType,
-        text: translate(`workspace.netsuite.advancedConfig.accountingMethods.values.${accountingMehtodType}`),
-        alternateText: translate(`workspace.netsuite.advancedConfig.accountingMethods.alternateText.${accountingMehtodType}`),
-        keyForList: accountingMehtodType,
-        isSelected: accountingMehtod === accountingMehtodType,
+    const config = policy?.connections?.netsuite?.options?.config;
+    const accountingMethod = config?.accountingMethod ?? CONST.NETSUITE_ACCOUNTING_METHODS.CASH;
+    const data: MenuListItem[] = Object.values(CONST.NETSUITE_ACCOUNTING_METHODS).map((accountingMethodType) => ({
+        value: accountingMethodType,
+        text: translate(`workspace.netsuite.advancedConfig.accountingMethods.values.${accountingMethodType}`),
+        alternateText: translate(`workspace.netsuite.advancedConfig.accountingMethods.alternateText.${accountingMethodType}`),
+        keyForList: accountingMethodType,
+        isSelected: accountingMethod === accountingMethodType,
     }));
 
     const headerContent = useMemo(

src/pages/workspace/accounting/netsuite/advanced/NetSuiteAdvancedPage.tsx

@@ -64,11 +64,11 @@ function NetSuiteAdvancedPage({policy}: WithPolicyConnectionsProps) {
     const menuItems: Array<MenuItemWithSubscribedSettings | ToggleItem | DividerLineItem> = [
         {
             type: 'menuitem',
-            title: autoSyncConfig?.autoSync.enabled ? translate('common.enabled') : translate('common.disabled'),
+            title: autoSyncConfig?.autoSync?.enabled ? translate('common.enabled') : translate('common.disabled'),
             description: translate('workspace.accounting.autoSync'),
             onPress: () => Navigation.navigate(ROUTES.POLICY_ACCOUNTING_NETSUITE_AUTO_SYNC.getRoute(policyID)),
             hintText: (() => {
-                if (!autoSyncConfig?.autoSync.enabled) {
+                if (!autoSyncConfig?.autoSync?.enabled) {
                     return undefined;
                 }
                 return translate(`workspace.netsuite.advancedConfig.accountingMethods.alternateText.${accountingMethod ?? CONST.NETSUITE_ACCOUNTING_METHODS.CASH}`);

src/pages/workspace/accounting/netsuite/advanced/NetSuiteAutoSyncPage.tsx

@@ -21,7 +21,7 @@ function NetSuiteAutoSyncPage({policy, route}: WithPolicyConnectionsProps) {
     const {translate} = useLocalize();
     const autoSyncConfig = policy?.connections?.netsuite?.config;
     const policyID = route.params.policyID ?? '-1';
-    const accountingMethod = policy?.connections?.netsuite?.options.config.accountingMethod;
+    const accountingMethod = policy?.connections?.netsuite?.options?.config?.accountingMethod;
 
     return (
         <AccessOrNotFoundWrapper
@@ -42,7 +42,7 @@ function NetSuiteAutoSyncPage({policy, route}: WithPolicyConnectionsProps) {
                     title={translate('workspace.accounting.autoSync')}
                     // wil be converted to translate and spanish translation too
                     subtitle={translate('workspace.accounting.autoSyncDescription')}
-                    isActive={!!autoSyncConfig?.autoSync.enabled}
+                    isActive={!!autoSyncConfig?.autoSync?.enabled}
                     wrapperStyle={[styles.pv2, styles.mh5]}
                     switchAccessibilityLabel={translate('workspace.netsuite.advancedConfig.autoSyncDescription')}
                     shouldPlaceSubtitleBelowSwitch
@@ -51,10 +51,14 @@ function NetSuiteAutoSyncPage({policy, route}: WithPolicyConnectionsProps) {
                     pendingAction={settingsPendingAction([CONST.NETSUITE_CONFIG.AUTO_SYNC], autoSyncConfig?.pendingFields)}
                     errors={ErrorUtils.getLatestErrorField(autoSyncConfig, CONST.NETSUITE_CONFIG.AUTO_SYNC)}
                 />
-                {!!autoSyncConfig?.autoSync.enabled && (
+                {!!autoSyncConfig?.autoSync?.enabled && (
                     <MenuItemWithTopDescription
-                        title={accountingMethod === CONST.NETSUITE_ACCOUNTING_METHODS.ACCRUAL ? 'Accural' : 'Cash'}
-                        description="When to export"
+                        title={
+                            accountingMethod === CONST.NETSUITE_ACCOUNTING_METHODS.ACCRUAL
+                                ? translate(`workspace.netsuite.advancedConfig.accountingMethods.values.${CONST.NETSUITE_ACCOUNTING_METHODS.ACCRUAL}`)
+                                : translate(`workspace.netsuite.advancedConfig.accountingMethods.values.${CONST.NETSUITE_ACCOUNTING_METHODS.CASH}`)
+                        }
+                        description={translate('workspace.netsuite.advancedConfig.accountingMethods.label')}
                         shouldShowRightIcon
                         onPress={() => Navigation.navigate(ROUTES.POLICY_ACCOUNTING_NETSUITE_ACCOUNTING_METHOD.getRoute(policyID))}
                     />