fix: library sorting persistence by adding missing useEffect dependencies (#1047)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lostb1t <168401+lostb1t@users.noreply.github.com>
Co-authored-by: lance chant <13349722+lancechant@users.noreply.github.com>
This commit is contained in:
Copilot
2025-09-08 16:15:23 +02:00
committed by GitHub
parent ceafdbf9ee
commit 0b0592a699

View File

@@ -77,7 +77,13 @@ const Page = () => {
} else {
_setSortBy([SortByOption.SortName]);
}
}, []);
}, [
libraryId,
sortOrderPreference,
sortByPreference,
_setSortOrder,
_setSortBy,
]);
const setSortBy = useCallback(
(sortBy: SortByOption[]) => {
@@ -87,7 +93,7 @@ const Page = () => {
}
_setSortBy(sortBy);
},
[libraryId, sortByPreference],
[libraryId, sortByPreference, setSortByPreference, _setSortBy],
);
const setSortOrder = useCallback(
@@ -101,7 +107,7 @@ const Page = () => {
}
_setSortOrder(sortOrder);
},
[libraryId, sortOrderPreference],
[libraryId, sortOrderPreference, setOderByPreference, _setSortOrder],
);
const nrOfCols = useMemo(() => {