-
Notifications
You must be signed in to change notification settings - Fork 3
Added support for providers implementing async load and clear methods. #16
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
base: master
Are you sure you want to change the base?
Added support for providers implementing async load and clear methods. #16
Conversation
lib/Storeit.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is too long.
76759d7 to
cefbf88
Compare
|
🎱 See comments. |
7e6e4d7 to
c596d42
Compare
lib/Storeit.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is too long.
c596d42 to
239f9a4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is too long.
239f9a4 to
5dd56fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line is too long.
|
Looks good to me. I'd like to @donavon to take a look when he can. |
|
👍 |
No description provided.