Refactor: fetch pending queue only in 1/N Safes

safe-global · Aug 7, 2023 · 3824772 · 3824772
1 parent beae29d
commit 3824772
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/hooks/usePendingTxs.ts b/src/hooks/usePendingTxs.ts
@@ -22,6 +22,15 @@ export const useHasPendingTxs = (): boolean => {
   return pendingIds.length > 0
 }
 
+/**
+ * Show unsigned pending queue only in 1/X Safes
+ */
+export const useShowUnsignedQueue = (): boolean => {
+  const { safe } = useSafeInfo()
+  const hasPending = useHasPendingTxs()
+  return safe.threshold === 1 && hasPending
+}
+
 export const usePendingTxsQueue = (): {
   page?: TransactionListPage
   error?: string

diff --git a/src/pages/transactions/queue.tsx b/src/pages/transactions/queue.tsx
@@ -6,10 +6,10 @@ import TxHeader from '@/components/transactions/TxHeader'
 import BatchExecuteButton from '@/components/transactions/BatchExecuteButton'
 import { Box } from '@mui/material'
 import { BatchExecuteHoverProvider } from '@/components/transactions/BatchExecuteButton/BatchExecuteHoverProvider'
-import { useHasPendingTxs, usePendingTxsQueue } from '@/hooks/usePendingTxs'
+import { usePendingTxsQueue, useShowUnsignedQueue } from '@/hooks/usePendingTxs'
 
 const Queue: NextPage = () => {
-  const hasPending = useHasPendingTxs()
+  const showPending = useShowUnsignedQueue()
 
   return (
     <>
@@ -25,7 +25,7 @@ const Queue: NextPage = () => {
         <main>
           <Box mb={4}>
             {/* Pending unsigned transactions */}
-            {hasPending && <PaginatedTxns useTxns={usePendingTxsQueue} />}
+            {showPending && <PaginatedTxns useTxns={usePendingTxsQueue} />}
 
             {/* The main queue of signed transactions */}
             <PaginatedTxns useTxns={useTxQueue} />