Skip to content

Added support for providers implementing async load and clear methods. - #16

Open
zjordan wants to merge 2 commits into
masterfrom
p/zj/add-support-for-providers-implementing-async-load-method
Open

Added support for providers implementing async load and clear methods.#16
zjordan wants to merge 2 commits into
masterfrom
p/zj/add-support-for-providers-implementing-async-load-method

Conversation

@zjordan

@zjordan zjordan commented Jan 12, 2016

Copy link
Copy Markdown

No description provided.

Comment thread lib/Storeit.js Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long.

@zjordan
zjordan force-pushed the p/zj/add-support-for-providers-implementing-async-load-method branch from 76759d7 to cefbf88 Compare January 12, 2016 17:24
@pbouzakis

Copy link
Copy Markdown
Collaborator

🎱 See comments.

@zjordan
zjordan force-pushed the p/zj/add-support-for-providers-implementing-async-load-method branch from 7e6e4d7 to c596d42 Compare January 12, 2016 19:44
Comment thread lib/Storeit.js Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long.

@zjordan
zjordan force-pushed the p/zj/add-support-for-providers-implementing-async-load-method branch from c596d42 to 239f9a4 Compare January 12, 2016 19:50
Comment thread lib/Storeit.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long.

@zjordan
zjordan force-pushed the p/zj/add-support-for-providers-implementing-async-load-method branch from 239f9a4 to 5dd56fa Compare January 12, 2016 20:00
Comment thread lib/Storeit.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long.

@pbouzakis

Copy link
Copy Markdown
Collaborator

Looks good to me. I'd like to @donavon to take a look when he can.

@donavon

donavon commented Feb 1, 2016

Copy link
Copy Markdown
Owner

👍

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.

4 participants