Skip to content

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.

2 participants