Fix bugs in payee table (#3768)

actualbudget · Nov 2, 2024 · 6666014 · 6666014
1 parent dc42504
commit 6666014
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/packages/desktop-client/src/components/payees/PayeeTable.tsx b/packages/desktop-client/src/components/payees/PayeeTable.tsx
@@ -12,7 +12,7 @@ import { type PayeeEntity } from 'loot-core/src/types/models';
 
 import { useSelectedItems } from '../../hooks/useSelected';
 import { View } from '../common/View';
-import { Table } from '../table';
+import { useTableNavigator, Table } from '../table';
 
 import { PayeeTableRow } from './PayeeTableRow';
 
@@ -46,9 +46,16 @@ export const PayeeTable = forwardRef<
     setHovered(id);
   }, []);
 
+  const tableNavigator = useTableNavigator(payees, item =>
+    item.transfer_acct == null
+      ? ['select', 'name', 'rule-count']
+      : ['rule-count'],
+  );
+
   return (
     <View style={{ flex: 1 }} onMouseLeave={() => setHovered(null)}>
       <Table
+        navigator={tableNavigator}
         ref={ref}
         items={payees}
         renderItem={({ item, editing, focusedField, onEdit }) => {

diff --git a/packages/desktop-client/src/components/payees/PayeeTableRow.tsx b/packages/desktop-client/src/components/payees/PayeeTableRow.tsx
@@ -133,6 +133,9 @@ export const PayeeTableRow = memo(
           focused={focusedField === 'select'}
           selected={selected}
           onSelect={e => {
+            if (payee.transfer_acct != null) {
+              return;
+            }
             dispatchSelected({
               type: 'select',
               id: payee.id,

diff --git a/upcoming-release-notes/3768.md b/upcoming-release-notes/3768.md
@@ -0,0 +1,6 @@
+---
+category: Bugfix
+authors: [jfdoming]
+---
+
+Fix bugs on payee management page