Skip to content
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

feat(storage): setEngine method to choose the storage option #6

Merged
merged 4 commits into from
Oct 17, 2016

Conversation

jgw96
Copy link
Contributor

@jgw96 jgw96 commented Sep 30, 2016

No description provided.

@jgw96
Copy link
Contributor Author

jgw96 commented Sep 30, 2016

The if was there to give the user an error if they did not choose one of the default ones, but i totally did not think about the fact that LocalForage accepts arbitrary engines. Ill change that.

@jgw96
Copy link
Contributor Author

jgw96 commented Sep 30, 2016

There we go, should be ready to go now.

@mlynch
Copy link
Contributor

mlynch commented Sep 30, 2016

One last nitpick :) Can we call ours setDriver to keep it the same as localForage? No need to rename it IMO

@mlynch mlynch merged commit 5974840 into master Oct 17, 2016
@mlynch
Copy link
Contributor

mlynch commented Oct 17, 2016

Thanks!

@mhartington mhartington deleted the feature/setDriver branch July 23, 2020 19:09
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.

None yet

2 participants