✨ show the 'import transactions' button even if accounts have banksync (

actualbudget#3615)
MikesGlitch · Oct 9, 2024 · 79f640c · 79f640c
1 parent f786bdc
commit 79f640c
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 27 deletions.
diff --git a/packages/desktop-client/src/components/accounts/Header.jsx b/packages/desktop-client/src/components/accounts/Header.jsx
@@ -287,38 +287,37 @@ export function AccountHeader({
           align="center"
           style={{ marginTop: 12 }}
         >
-          {((account && !account.closed) || canSync) && (
+          {canSync && (
             <Button
               variant="bare"
-              onPress={canSync ? onSync : onImport}
-              isDisabled={canSync && isServerOffline}
+              onPress={onSync}
+              isDisabled={isServerOffline}
             >
-              {canSync ? (
-                <>
-                  <AnimatedRefresh
-                    width={13}
-                    height={13}
-                    animating={
-                      account
-                        ? accountsSyncing.includes(account.id)
-                        : accountsSyncing.length > 0
-                    }
-                    style={{ marginRight: 4 }}
-                  />{' '}
-                  {isServerOffline ? t('Bank Sync Offline') : t('Bank Sync')}
-                </>
-              ) : (
-                <>
-                  <SvgDownloadThickBottom
-                    width={13}
-                    height={13}
-                    style={{ marginRight: 4 }}
-                  />{' '}
-                  <Trans>Import</Trans>
-                </>
-              )}
+              <AnimatedRefresh
+                width={13}
+                height={13}
+                animating={
+                  account
+                    ? accountsSyncing.includes(account.id)
+                    : accountsSyncing.length > 0
+                }
+                style={{ marginRight: 4 }}
+              />{' '}
+              {isServerOffline ? t('Bank Sync Offline') : t('Bank Sync')}
+            </Button>
+          )}
+
+          {account && !account.closed && (
+            <Button variant="bare" onPress={onImport}>
+              <SvgDownloadThickBottom
+                width={13}
+                height={13}
+                style={{ marginRight: 4 }}
+              />{' '}
+              <Trans>Import</Trans>
             </Button>
           )}
+
           {!showEmptyMessage && (
             <Button variant="bare" onPress={onAddTransaction}>
               <SvgAdd width={10} height={10} style={{ marginRight: 3 }} />

diff --git a/upcoming-release-notes/3615.md b/upcoming-release-notes/3615.md
@@ -0,0 +1,6 @@
+---
+category: Enhancements
+authors: [MatissJanis]
+---
+
+Show the "import transactions" button even if accounts have bank-sync enabled.