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

Fixes an issue when you pass an url directly. #34

Merged
merged 2 commits into from
Oct 21, 2013
Merged

Fixes an issue when you pass an url directly. #34

merged 2 commits into from
Oct 21, 2013

Conversation

mweibel
Copy link
Contributor

@mweibel mweibel commented Oct 7, 2013

This is nice for example when you fetch og.likes with
a next URL in the response which has already all the necessary fields in it.
With this fix you're able to pass a fully constructed URL (so,
in this case, it's merely a wrapper around request but it
helps when you can use the same API).

This happens for example when you fetch og.likes with
a next URL which is already with all the necessary fields in it.
With this fix you're able to pass a fully constructed URL (so,
in this case, it's merely a wrapper around `request` but it
helps when you can use the same API).
@mweibel
Copy link
Contributor Author

mweibel commented Oct 14, 2013

Ah, I see what you mean.
Yes this does fix this indeed.

I added now a bit of documentation although I'm not particularly happy about it :/

criso added a commit that referenced this pull request Oct 21, 2013
Fixes an issue when you pass an url directly.
@criso criso merged commit 73d79ef into criso:master Oct 21, 2013
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.

2 participants