fix(tables): symmetric guarded onSettled across row write mutations

waleedlatif1 · waleedlatif1 · commit a0974d6980aa · 2026-05-13T19:15:36.000-07:00
diff --git a/apps/sim/hooks/queries/tables.ts b/apps/sim/hooks/queries/tables.ts
@@ -594,11 +594,13 @@ export function useUpdateTableRow({ workspaceId, tableId }: RowMutationContext)
           queryClient.setQueryData(queryKey, data)
         }
       }
+      if (isValidationError(error)) return
+      toast.error(error.message, { duration: 5000 })
+    },
+    onSettled: () => {
       if (queryClient.isMutating({ mutationKey: tableKeys.rowWrites(tableId) }) === 1) {
         invalidateRowData(queryClient, tableId)
       }
-      if (isValidationError(error)) return
-      toast.error(error.message, { duration: 5000 })
     },
   })
 }

Original file line number	Diff line number	Diff line change
`@@ -594,11 +594,13 @@ export function useUpdateTableRow({ workspaceId, tableId }: RowMutationContext)`
`594`	`594`	`queryClient.setQueryData(queryKey, data)`
`595`	`595`	`}`
`596`	`596`	`}`
	`597`	`+ if (isValidationError(error)) return`
	`598`	`+ toast.error(error.message, { duration: 5000 })`
	`599`	`+ },`
	`600`	`+ onSettled: () => {`
`597`	`601`	`if (queryClient.isMutating({ mutationKey: tableKeys.rowWrites(tableId) }) === 1) {`
`598`	`602`	`invalidateRowData(queryClient, tableId)`
`599`	`603`	`}`
`600`		`- if (isValidationError(error)) return`
`601`		`- toast.error(error.message, { duration: 5000 })`
`602`	`604`	`},`
`603`	`605`	`})`
`604`	`606`	`}`