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

localForage support #149

Open
siddo420 opened this Issue Aug 29, 2015 · 1 comment

Comments

Projects
None yet
2 participants
@siddo420

siddo420 commented Aug 29, 2015

Can I use localForage instead of localstorage?

What part of code needs to be changed to support localForage?

@nilbus

This comment has been minimized.

Show comment
Hide comment
@nilbus

nilbus Aug 29, 2015

Owner

Your goal is to use IndexedDB or WebSQL, right? Check out the async branch (and #73), which does that already. It uses sticky.js instead of localForage for the adapter, but if that matters to you, it would be trivial to write one in backbone.dualstorage.adapters.coffee.

The async branch has some failing tests related to how promises act in some situations, but it should otherwise work fine. If you think you'd like to address those issues before I get to it, have at it.

Owner

nilbus commented Aug 29, 2015

Your goal is to use IndexedDB or WebSQL, right? Check out the async branch (and #73), which does that already. It uses sticky.js instead of localForage for the adapter, but if that matters to you, it would be trivial to write one in backbone.dualstorage.adapters.coffee.

The async branch has some failing tests related to how promises act in some situations, but it should otherwise work fine. If you think you'd like to address those issues before I get to it, have at it.

@nilbus nilbus changed the title from localforage to localForage support Aug 29, 2015

@nilbus nilbus added Idea async labels Aug 29, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment