sinvo/docmost
1
0
Fork 0
mirror of https://github.com/docmost/docmost.git synced 2026-09-01 03:15:32 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(base): stabilize properties identity to break render loop

Browse Source
This commit is contained in:
Philipinho
2026-04-18 18:48:41 +01:00
parent 5a4d10081d
commit 5c11e59128
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/client/src/features/base/hooks/use-base-table.ts
+5 -1
View File
@@ -185,7 +185,11 @@ export function useBaseTable(
const updateViewMutation = useUpdateViewMutation(); const updateViewMutation = useUpdateViewMutation();
const persistTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null); const persistTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
const properties = base?.properties ?? []; // `base?.properties ?? []` minted a fresh `[]` every render while the
// base query was loading, which invalidated every downstream memo and
// tripped the setState-in-useEffect pairs below → "Maximum update
// depth exceeded". Memoize so the identity is stable.
const properties = useMemo(() => base?.properties ?? [], [base?.properties]);
const viewConfig = activeView?.config; const viewConfig = activeView?.config;
const columns = useMemo( const columns = useMemo(