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

Add ability to retry cURL connections if they fail (for any reason). … #195

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

RobWiddick
Copy link
Contributor

…Fixes issue #194

@RobWiddick
Copy link
Contributor Author

PSR-2 compliant... whoops.

I will also check over v3 to help out! Thanks!

@RobWiddick
Copy link
Contributor Author

Pull request now includes fixes for 3 other issues regarding available scopes

@RobWiddick
Copy link
Contributor Author

I just realized that the last commit may be going into v3.. but this may be good for transitional users.

It's important to note that those new scopes won't work unless the Instagram app itself is upgraded/reviewed. It might be good to put that in the documentation somewhere.

@zack-hable
Copy link

zack-hable commented Dec 9, 2016

@robertark can you please verify that this is still working? I've tried to merge your changes into my own (with added scopes, x-rate-limit fix and added subscription calls from the various other PRs on here) and with your's (even just by downloading your file) gives errors when returning comments/likes.

edit: only affects retrieving comments
stdClass Object ( [meta] => stdClass Object ( [error_type] => OAuthAccessTokenException [code] => 400 [error_message] => The access_token provided is invalid. ) )

@zack-hable
Copy link

seems this needs to be changed
public function getMediaComments($id)
{
return $this->_makeCall('media/' . $id . '/comments', true);
}

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.

None yet

2 participants