ack PR comments

This commit is contained in:
waleed
2026-02-14 17:43:47 -08:00
parent 9f98f59cc7
commit ca0105aceb
3 changed files with 15 additions and 12 deletions

View File

@@ -371,9 +371,7 @@ function DashboardInner({ stats, isLoading, error }: DashboardProps) {
[toggleWorkflowId] [toggleWorkflowId]
) )
useEffect(() => { lastAnchorIndicesRef.current = lastAnchorIndices
lastAnchorIndicesRef.current = lastAnchorIndices
}, [lastAnchorIndices])
/** /**
* Handles segment click for selecting time segments. * Handles segment click for selecting time segments.

View File

@@ -234,8 +234,7 @@ const WorkflowOutputSection = memo(
</div> </div>
) )
}, },
(prev, next) => (prev, next) => prev.output === next.output
prev.output === next.output || JSON.stringify(prev.output) === JSON.stringify(next.output)
) )
interface LogDetailsProps { interface LogDetailsProps {

View File

@@ -230,13 +230,19 @@ export default function Logs() {
useEffect(() => { useEffect(() => {
selectedLogIdRef.current = selectedLogId selectedLogIdRef.current = selectedLogId
}, [selectedLogId]) }, [selectedLogId])
logsRefetchRef.current = logsQuery.refetch useEffect(() => {
activeLogRefetchRef.current = activeLogQuery.refetch logsRefetchRef.current = logsQuery.refetch
logsQueryRef.current = { }, [logsQuery.refetch])
isFetching: logsQuery.isFetching, useEffect(() => {
hasNextPage: logsQuery.hasNextPage ?? false, activeLogRefetchRef.current = activeLogQuery.refetch
fetchNextPage: logsQuery.fetchNextPage, }, [activeLogQuery.refetch])
} useEffect(() => {
logsQueryRef.current = {
isFetching: logsQuery.isFetching,
hasNextPage: logsQuery.hasNextPage ?? false,
fetchNextPage: logsQuery.fetchNextPage,
}
}, [logsQuery.isFetching, logsQuery.hasNextPage, logsQuery.fetchNextPage])
useEffect(() => { useEffect(() => {
const timers = refreshTimersRef.current const timers = refreshTimersRef.current