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 instagram friend lists scope #458

Merged
merged 2 commits into from
Feb 11, 2016
Merged

Conversation

sajjadrad
Copy link
Contributor

No description provided.

@sajjadrad
Copy link
Contributor Author

@elliotchance
Copy link
Collaborator

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.


src/OAuth/OAuth2/Service/Instagram.php, line 24 [r1] (raw file):
Why did you pick SCOPE_FRIEND_LIST instead of SCOPE_FOLLOWER_LIST like the convention of the others?


Comments from the review on Reviewable.io

@sajjadrad
Copy link
Contributor Author

@elliotchance because it's not just follower_list, it will get following lists too.

@elliotchance
Copy link
Collaborator

Review status: all files reviewed at latest revision, 1 unresolved discussion.


src/OAuth/OAuth2/Service/Instagram.php, line 24 [r1] (raw file):
Please reply on the reviewable: https://reviewable.io/reviews/lusitanian/phpoauthlib/458

I'm referring to the naming convention not what the actual scope returns. The name of the constant should be indicative of the value it represents like all the other constants.

Following on from what you said it would be good to put that as a comment on the constant since that's not clear.


Comments from the review on Reviewable.io

@sajjadrad
Copy link
Contributor Author

Const name changed.

@elliotchance
Copy link
Collaborator

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

elliotchance added a commit that referenced this pull request Feb 11, 2016
Add instagram friend lists scope
@elliotchance elliotchance merged commit 8a97b77 into daviddesberg:master Feb 11, 2016
@elliotchance
Copy link
Collaborator

Thanks @sajjadrad! https://github.com/Lusitanian/PHPoAuthLib/releases/tag/v0.8.7


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from the review on Reviewable.io

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