Refactor infiniteScoll to use debouncing (#5999)

Same as https://github.com/twentyhq/twenty/pull/5996 but with
useDebounced as asked in review
This commit is contained in:
Charles Bochet 2024-06-24 13:45:07 +02:00 committed by GitHub
parent 2e4ba9ca7b
commit 498e4ff6ba
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -1,5 +1,6 @@
import { useEffect } from 'react'; import { useEffect } from 'react';
import { useRecoilValue } from 'recoil'; import { useRecoilValue } from 'recoil';
import { useDebouncedCallback } from 'use-debounce';
import { useLoadRecordIndexTable } from '@/object-record/record-index/hooks/useLoadRecordIndexTable'; import { useLoadRecordIndexTable } from '@/object-record/record-index/hooks/useLoadRecordIndexTable';
import { useRecordTableStates } from '@/object-record/record-table/hooks/internal/useRecordTableStates'; import { useRecordTableStates } from '@/object-record/record-table/hooks/internal/useRecordTableStates';
@ -41,14 +42,20 @@ export const RecordTableBodyEffect = ({
} }
}, [records, totalCount, setRecordTableData, loading]); }, [records, totalCount, setRecordTableData, loading]);
const fetchMoreDebouncedIfRequested = useDebouncedCallback(async () => {
// We are debouncing here to give the user some room to scroll if they want to within this throttle window
await fetchMoreObjects();
}, 100);
useEffect(() => { useEffect(() => {
// We are adding a setTimeout here to give the user some room to scroll if they want to within this throttle window if (!isFetchingMoreObjects && tableLastRowVisible) {
setTimeout(async () => { fetchMoreDebouncedIfRequested();
if (!isFetchingMoreObjects && tableLastRowVisible) { }
await fetchMoreObjects(); }, [
} fetchMoreDebouncedIfRequested,
}, 100); isFetchingMoreObjects,
}, [fetchMoreObjects, isFetchingMoreObjects, tableLastRowVisible]); tableLastRowVisible,
]);
return <></>; return <></>;
}; };