Skip to content
This repository was archived by the owner on Dec 2, 2024. It is now read-only.

Unwrap #88

Merged
merged 7 commits into from
May 25, 2018
Merged

Unwrap #88

merged 7 commits into from
May 25, 2018

Conversation

vweevers
Copy link
Member

@vweevers vweevers commented May 24, 2018 •
edited
Loading

  • get, put, del
  • batch
  • iterator
  • open
  • close

Closes #46.

@vweevers vweevers self-assigned this May 24, 2018
@vweevers vweevers requested a review from ralphtheninja May 25, 2018 13:42
@vweevers vweevers changed the title Unwrap [WIP] Unwrap May 25, 2018
@vweevers
Copy link
Member Author

Supersedes #55 (you can now pass a prefix option to the contructor) and I think #53 as well (it seems the use case there is the same: specify a custom prefix).

@vweevers
Copy link
Member Author

This changes the default prefix and removes options that IDBWrapper had but we don't, so I'm adding the upgrading label.

@vweevers vweevers added the upgrade guide Improvements or additions to upgrade guide(s) label May 25, 2018
@vweevers vweevers merged commit b1c01a9 into master May 25, 2018
@vweevers vweevers deleted the unwrap branch May 25, 2018 14:24
This was referenced May 25, 2018
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ralphtheninja ralphtheninja ralphtheninja approved these changes

Assignees

@vweevers vweevers

Labels
upgrade guide Improvements or additions to upgrade guide(s)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

swap out idbwrapper with straight indexeddb code
2 participants
@vweevers @ralphtheninja