useLocalStorage and useSessionStorage missing dependencies

[israelins85]

Fixes #5911

[rtivital]

What real problem do you have that this PR fixes?

[israelins85]

When key is changed the value is not loaded.

[israelins85]

fixes #5911

[rtivital]

Thanks!

[gl3nn]

@rtivital @israelins85 This PR broke all of our usages of it and thus can't update to 7.7.1. Would be nice in the future to show example code on what your issue was and why this would fix it.

[rtivital]

It would be reverted in next patch

[israelins85]

It would be reverted in next patch

What's happened?

[israelins85]

@rtivital @israelins85 This PR broke all of our usages of it and thus can't update to 7.7.1. Would be nice in the future to show example code on what your issue was and why this would fix it.

https://codesandbox.io/p/sandbox/mantine-react-template-forked-wk42tp?file=%2Fsrc%2FApp.tsx%3A6%2C2

What's broken?

[israelins85]

I use localStorage to store data from many devices key is like ${uuid}/Sensors and when user changes the current uuid the data of the current need be loaded...

[israelins85]

#6019

[icflorescu]

@rtivital @israelins85 @gl3nn this also broke Mantine DataTable on Mantine 7.7.1, see this issue: icflorescu/mantine-datatable#564 😑

We're using the useLocalStorage hook to persist column resizing/ordering state, and it gets triggered infinitely with the same values:

Would be nice to see #6019 pulled...

[rtivital]

I've reverted it, the fix will be available in next patch