Merge pull request #47290 from nkdengineer/fix/46675

fix: Preferred exporter check-mark is missing
Expensify · Aug 14, 2024 · 29fb6bb · 29fb6bb
2 parents 3b48840 + 8490915
commit 29fb6bb
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/pages/workspace/accounting/netsuite/export/NetSuitePreferredExporterSelectPage.tsx b/src/pages/workspace/accounting/netsuite/export/NetSuitePreferredExporterSelectPage.tsx
@@ -55,7 +55,7 @@ function NetSuitePreferredExporterSelectPage({policy}: WithPolicyConnectionsProp
                 value: exporter.email,
                 text: exporter.email,
                 keyForList: exporter.email,
-                isSelected: config?.exporter === exporter.email,
+                isSelected: (config?.exporter ?? policyOwner) === exporter.email,
             });
             return options;
         }, []);

diff --git a/src/pages/workspace/accounting/qbo/export/QuickbooksPreferredExporterConfigurationPage.tsx b/src/pages/workspace/accounting/qbo/export/QuickbooksPreferredExporterConfigurationPage.tsx
@@ -45,7 +45,7 @@ function QuickbooksPreferredExporterConfigurationPage({policy}: WithPolicyConnec
                     value: exporter.email,
                     text: exporter.email,
                     keyForList: exporter.email,
-                    isSelected: exportConfiguration?.exporter === exporter.email,
+                    isSelected: (exportConfiguration?.exporter ?? policy?.owner) === exporter.email,
                 });
                 return options;
             }, []),

diff --git a/src/pages/workspace/accounting/xero/export/XeroPreferredExporterSelectPage.tsx b/src/pages/workspace/accounting/xero/export/XeroPreferredExporterSelectPage.tsx
@@ -58,11 +58,11 @@ function XeroPreferredExporterSelectPage({policy}: WithPolicyConnectionsProps) {
                 value: exporter.email,
                 text: exporter.email,
                 keyForList: exporter.email,
-                isSelected: config?.export?.exporter === exporter.email,
+                isSelected: (config?.export?.exporter ?? policyOwner) === exporter.email,
             });
             return options;
         }, []);
-    }, [config?.export?.exporter, exporters, policyOwner, currentUserLogin]);
+    }, [policyOwner, exporters, currentUserLogin, config?.export?.exporter]);
 
     const selectExporter = useCallback(
         (row: CardListItem) => {