-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Add localestorage support #22
Comments
Currently not possible without a rework ( I'm working on it though, as one with Not published, but see https://github.com/ZaneHannanAU/db-dot-json for more information. |
@ZaneHannanAU Great work on implementing the Promises :) I admit I gave up on Async as it was overkill for the usage I had of the lib at the time (hence the decision to use Sync filesystem calls). |
@Belphemur The biggest issue with mine right now is that it doesn't work at all. As in, not even the tests work lol Anyway, once you go promise for one thing; everything else must become a promise. |
@ZaneHannanAU Yep, Promises are a double edge sword. One that I didn't wanted to impose on the user. Also at the time I started the lib, there were easily 5-6 differents promise lib... |
@Belphemur I think an easier solution would be to:
I can do it in a PR... I need this feature 😁 |
To be able to use it in browser easily and rely on the storage mechanism there.
The text was updated successfully, but these errors were encountered: