useLocalStorage not working with updater function

[acontreras89]

useLocalStorage is not updating the value properly when passing an updater function. This is due to state not being included in the useCallback deps for the setter function (line 82 below).

react-use/src/useLocalStorage.ts

Lines 59 to 83 in 325f5bd

	const set: Dispatch<SetStateAction<T | undefined>> = useCallback(
	(valOrFunc) => {
	try {
	const newState =
	typeof valOrFunc === 'function' ? (valOrFunc as Function)(state) : valOrFunc;
	if (typeof newState === 'undefined') return;
	let value: string;
	if (options)
	if (options.raw)
	if (typeof newState === 'string') value = newState;
	else value = JSON.stringify(newState);
	else if (options.serializer) value = options.serializer(newState);
	else value = JSON.stringify(newState);
	else value = JSON.stringify(newState);
	localStorage.setItem(key, value);
	setState(deserializer(value));
	} catch {
	// If user is in private mode or has storage restriction
	// localStorage can throw. Also JSON.stringify can throw.
	}
	},
	[key, setState]
	);

(I may be able to open a PR for this if I find some time to spare this or next week.)

[acontreras89]

This seems to be the same problem reported in #2273, although using pretty vague title and description. Also, PR #2277 aims at solving this, although it might need some extra work (and tests).