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 pagination to list API endpoints that do not have it yet #18082

Open
15 tasks
qwerty287 opened this issue Dec 23, 2021 · 3 comments
Open
15 tasks

Add pagination to list API endpoints that do not have it yet #18082

qwerty287 opened this issue Dec 23, 2021 · 3 comments
Labels
modifies/api This PR adds API routes or modifies them type/summary This issue aggregates a bunch of other issues

Comments

@qwerty287
Copy link
Contributor

qwerty287 commented Dec 23, 2021

  • /repos/{owner}/{repo}/issues/{index}/labels
  • /repos/{owner}/{repo}/issues/comments/{id}/reactions
  • /repos/{owner}/{repo}/branch_protections
  • /repos/{owner}/{repo}/contents
  • /repos/{owner}/{repo}/git/refs
  • /repos/{owner}/{repo}/git/refs/{ref}
  • /repos/{owner}/{repo}/hooks/git
  • /repos/{owner}/{repo}/issue_templates
  • /repos/{owner}/{repo}/releases/{id}/assets
  • /repos/{owner}/{repo}/reviewers
  • /repos/{owner}/{repo}/teams
  • /user/emails
  • /users/{username}/heatmap
  • https://codeberg.org/api/swagger#/issue/issueGetComments (add page and limit params)
  • /repos/{owner}/{repo}/git/trees/{sha} (use limit param instead of per_page)
@noerw noerw added modifies/api This PR adds API routes or modifies them type/enhancement An improvement of existing functionality labels Dec 24, 2021
@lunny lunny added the type/summary This issue aggregates a bunch of other issues label Dec 31, 2021
@qwerty287
Copy link
Contributor Author

There is also one endpoints that only support since/before params, but not page/limit`:

https://codeberg.org/api/swagger#/issue/issueGetComments

@qwerty287
Copy link
Contributor Author

And /repos/{owner}/{repo}/git/trees/{sha} uses per_page instead oft limit, should be deprecated.

@lunny
Copy link
Member

lunny commented Dec 31, 2021

And /repos/{owner}/{repo}/git/trees/{sha} uses per_page instead oft limit, should be deprecated.

I updated your issue so that we can follow them easier. :)

@lunny lunny removed the type/enhancement An improvement of existing functionality label Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modifies/api This PR adds API routes or modifies them type/summary This issue aggregates a bunch of other issues
Projects
None yet
Development

No branches or pull requests

3 participants