Option for not persisting in local storage? #87

woigl · 2023-09-23T16:32:11Z

First of all, let me express my gratitude for this library.

I propose a feature to deactivate persisting in the local storage by not calling the respective setData(...). The reason is that I don't need the data persistent as I handle all the rest on the backend. You may have a better idea to achieve this in compliance with your vision of this library.

Would it be okay if I prepare the code and initiate a pull request, or you don't want this enhancement?

Thanks in advance for your consideration.

tasoskakour · 2023-09-24T12:23:51Z

Hey @woigl

That wouldn't be a bad idea actually. Although not something urgent, it would be maybe a "nice-to-have" 🙂

I believe though, that if we go with this approach it would be more beneficial to write our own hook for persisting the state and not relying on the use-local-storage-state hook that we currently rely on.

It could be something like:

const [state, setState] =  usePersistedState(
   storageKey,
   defaultState, 
   { persist: true | false, storage: window.localStorage | window.sessionStorage }
);

With persist: false a user could skip the persistence whatsoever.
With storage: window.sessionStorage a user could persist to sessionStorage instead of localStorage

tasoskakour changed the title ~~Option for not persisting in session storage?~~ Option for not persisting in local storage? Sep 24, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Option for not persisting in local storage? #87

Option for not persisting in local storage? #87

Uh oh!

Option for not persisting in local storage? #87

Option for not persisting in local storage? #87

Comments

Uh oh!

Uh oh!