fix(logs): guard fetchNextPage call until query has succeeded

waleedlatif1 · waleedlatif1 · commit 48bdbe58319a · 2026-03-30T18:25:16.000-07:00
Add logsQuery.status === 'success' to the fetchNextPage branch so it
mirrors the clear branch. On mount the query is disabled (isFetching is
false, status is pending), causing the effect to call fetchNextPage()
before the query is initialized — now both branches require success.
diff --git a/apps/sim/app/workspace/[workspaceId]/logs/logs.tsx b/apps/sim/app/workspace/[workspaceId]/logs/logs.tsx
@@ -437,7 +437,7 @@ export default function Logs() {
       dispatch({ type: 'TOGGLE_LOG', logId: found.id })
     } else if (!logsQuery.hasNextPage && logsQuery.status === 'success') {
       pendingExecutionIdRef.current = null
-    } else if (!logsQuery.isFetching) {
+    } else if (!logsQuery.isFetching && logsQuery.status === 'success') {
       logsQueryRef.current.fetchNextPage()
     }
   }, [sortedLogs, logsQuery.hasNextPage, logsQuery.isFetching, logsQuery.status])

Original file line number	Diff line number	Diff line change
`@@ -437,7 +437,7 @@ export default function Logs() {`
`437`	`437`	`dispatch({ type: 'TOGGLE_LOG', logId: found.id })`
`438`	`438`	`} else if (!logsQuery.hasNextPage && logsQuery.status === 'success') {`
`439`	`439`	`pendingExecutionIdRef.current = null`
`440`		`- } else if (!logsQuery.isFetching) {`
	`440`	`+ } else if (!logsQuery.isFetching && logsQuery.status === 'success') {`
`441`	`441`	`logsQueryRef.current.fetchNextPage()`
`442`	`442`	`}`
`443`	`443`	`}, [sortedLogs, logsQuery.hasNextPage, logsQuery.isFetching, logsQuery.status])`