feat: IndexedDB support

joshnuss / svelte-persisted-store

A Svelte store that persists to localStorage

MIT License

994 stars 41 forks source link

feat: IndexedDB support #273

Open intercepted16 opened 3 months ago

intercepted16 commented 3 months ago

Implement IndexedDB support, useful for storing larger amounts of data.

The persisted function is now deprecated, but remains functional
Instead of a storage option, there are now three seperate functions; localState, sessionState and indexedDBState
The localState and sessionState functions remain the same, aside from the storage parameter
indexedDBState is asynchronous and uses async/await (or callback)
Some refactoring was done, specifically with types

PS: persisted remains functional with the same API

webJose commented 3 months ago

@intercepted16 thanks for the udpated PR, but please, don't continue unless @joshnuss says so. I just expressed my opinion in the form of a PR review. That's it. It is unclear to me if @joshnuss is even agreeable to adding this feature.

intercepted16 commented 3 months ago

@webJose Your changes were small, so I figured implementing them wouldn't hurt.

intercepted16 commented 2 months ago

@joshnuss Are you willing to add this feature? It's been a while with no response.

joshnuss commented 1 month ago

Hi, I do t have a good answer right now.

The next priority for the project is adding support for runes, so I'm not sure we should add more features right now.

Is this an important requirement you're waiting on?

intercepted16 commented 1 month ago

@joshnuss Yes, I'd prefer to use the library for this; currently I'm using a custom solution. Maybe, as breaking changes are being implemented anyway, you could include this pull request in the runes change?