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

move request pooling into hoodie.js core #118

Closed
gr2m opened this issue Aug 12, 2013 · 2 comments
Closed

move request pooling into hoodie.js core #118

gr2m opened this issue Aug 12, 2013 · 2 comments

Comments

@gr2m
Copy link
Member

gr2m commented Aug 12, 2013

I made withSingleRequest & withPreviousRequestsAborted for request pooling in the hoodie.account pooling.

I think withSingleRequest should be the default behavior of GETs / DELETEs and withPreviousRequestsAborted the default behavior for PUTs / POSTs in hoodie.request.

That means if I call hoodie.request('GET', '/_session') several times before the first request finishes, the subsequent calls don't send another request, but simply get the promise of the first.

@svnlto
Copy link
Member

svnlto commented Aug 12, 2013

+1

@gr2m
Copy link
Member Author

gr2m commented May 7, 2016

closing in favor of hoodiehq/hoodie-client#81

@gr2m gr2m closed this as completed May 7, 2016
@gr2m gr2m removed the later label May 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants