Skip to content

feat: separate localStorage and Map APIs #100

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 24, 2019
Merged

Conversation

cyrilletuzi
Copy link
Owner

@cyrilletuzi cyrilletuzi commented Apr 22, 2019

  • 2 distinct services, one localStorage-like, the other Map-like
  • deprecate Map-like API which was in LocalStorage
  • keys() should be iterating in StorageMap
  • re-investigate storing null / undefined
  • doc
  • tests

@cyrilletuzi cyrilletuzi self-assigned this Apr 22, 2019
@cyrilletuzi cyrilletuzi added this to the v8.0 milestone Apr 22, 2019
@cyrilletuzi cyrilletuzi merged commit 99c8ce8 into v8beta Apr 24, 2019
@cyrilletuzi cyrilletuzi deleted the abstractstorage branch April 24, 2019 13:49
cyrilletuzi added a commit that referenced this pull request May 28, 2019
* feat: new JSONSchema, and shortcuts and auto-inference for basic types
* feat: disallow null or undefined in setItem
* feat: allow custom indexeddb database and store names (#78)
* feat: better error management and move idb fallback to main service
* internal: reduce surface api
* fix: stringify all keys for interoperability
* fix: better error management
* feat: new getItem() signature (#79)
* fix: catch stringify error
* fix: catch quota error on localStorage setItem
* fix: catch missing store error
* fix: remove getUnsafeItem()
* feat: new IDB storing behavior with no wrapping (#82)
* feat: new prefix token for interop
* feat: removeItem when storing null or undefined (#93)
* feat: remove xSubscribe() methods (#96)
* feat: separate localStorage and Map APIs (#100)
* feat: iterating keys()
* fix: deprecate xSubscribe
* feat: rxjs 6.4 as minimum
* fix: use openKeyCursor() when possible for perf
* feat: schema option in set()
* fix: throw when storing data that can't be serialized in localStorage (#104)
* feat: engine and database info + opt-in for unwraped values in idb + new options system (#107)
* fix: new interface for new options system
* fix: test and doc about blob refused in idb in safari private
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant