Skip to content

Add ability to define parameters for /users/{user}/starred #536

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

Merged
merged 2 commits into from
Mar 23, 2017
Merged

Add ability to define parameters for /users/{user}/starred #536

merged 2 commits into from
Mar 23, 2017

Conversation

j0k3r
Copy link
Contributor

@j0k3r j0k3r commented Mar 7, 2017

Following the documentation we can define few parameters to the request:

  • page (was already available in the lib)
  • per_page
  • sort
  • direction

I also fixed a small typo.

@j0k3r j0k3r changed the title Add ability to define parameter for /users/{user}/starred Add ability to define parameters for /users/{user}/starred Mar 7, 2017
j0k3r added 2 commits March 7, 2017 11:59
Following the documentation (https://developer.github.com/v3/activity/starring/#list-repositories-being-starred) we can define few parameters to the request:
- page (was already available in the lib)
- per_page
- sort
- direction
Copy link
Collaborator

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. It looks good.

I cannot find the docs saying page, per_page and direction exists. (Looked here: https://developer.github.com/v3/activity/starring/) Can you please link me that docs?

@j0k3r
Copy link
Contributor Author

j0k3r commented Mar 7, 2017

I put the link in the PR description 🙂
https://developer.github.com/v3/activity/starring/#list-repositories-being-starred

That doc doesn't mention the per_page parameter but as all list api calls the page & per_page exists.
https://developer.github.com/guides/traversing-with-pagination/#basics-of-pagination

Copy link
Collaborator

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your PR

@Nyholm Nyholm merged commit 38301cc into KnpLabs:master Mar 23, 2017
@j0k3r j0k3r deleted the user-starred-parameters branch March 23, 2017 08:36
@j0k3r
Copy link
Contributor Author

j0k3r commented Mar 23, 2017

Thanks! Too bad it's not in 2.1.0

@Nyholm
Copy link
Collaborator

Nyholm commented Mar 23, 2017

2.2 will come today (or worst case this week).

@Nyholm Nyholm added this to the Release 2.2 milestone Mar 24, 2017
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