Title: Add pagination to search endpoints by alaycock · Pull Request #583 · github-tools/github · GitHub
Open Graph Title: Add pagination to search endpoints by alaycock · Pull Request #583 · github-tools/github
X Title: Add pagination to search endpoints by alaycock · Pull Request #583 · github-tools/github
Description: Changes: Updated the jsdoc to correctly document the Github.search function. If a page option is passed into _requestAllPages, it will only fetch that single page, instead of attempting to fetch all of the pages. Removed the results parameter from _requestAllPages, since that is only meant to be used internally anyway. It's trivial to re-introduce, so let me know if this makes sense to do, since this could break some backwards compatibility if folks are using an unsupported argument. Added a test for pagination, and a new fixture for that test. Fixed a number of other fixtures that didn't seem to have the correct page number. Changed the fixtures to use rawHeader instead of header, since the tests were not paginating. This was introduced when the spec for fixture files changed from nock@7 to nock@8. Let me know if this is the right approach. I've only run tests for the search endpoints (due to lack of access to the testing repo), so I'm not sure if this is going to introduce errors elsewhere. Closes #406 This could be a potential solution for #460, although introducing a limit could be an alternate solution.
Open Graph Description: Changes: Updated the jsdoc to correctly document the Github.search function. If a page option is passed into _requestAllPages, it will only fetch that single page, instead of attempting to fetch a...
X Description: Changes: Updated the jsdoc to correctly document the Github.search function. If a page option is passed into _requestAllPages, it will only fetch that single page, instead of attempting to fetch a...
Opengraph URL: https://github.com/github-tools/github/pull/583
X: @github
Domain: patch-diff.githubusercontent.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:0b91485a-5e93-68a6-b0db-e9b187f4faea |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 8542:1F10E2:C4A98E:1162E95:697258F8 |
| html-safe-nonce | 6a8ca6d8e381a0113894286708c052db980d924d9c0cb767fb3379f6b70c062a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4NTQyOjFGMTBFMjpDNEE5OEU6MTE2MkU5NTo2OTcyNThGOCIsInZpc2l0b3JfaWQiOiIyMjAxMzU2NDU5NjQ5OTQ3ODk3IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | ce11a95e5fb47fe4316048d3801c41c96b7fba1331454ef8d054d8b5d1aa49f4 |
| hovercard-subject-tag | pull_request:315130167 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/github-tools/github/pull/583/files |
| twitter:image | https://avatars.githubusercontent.com/u/894797?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/894797?s=400&v=4 |
| og:image:alt | Changes: Updated the jsdoc to correctly document the Github.search function. If a page option is passed into _requestAllPages, it will only fetch that single page, instead of attempting to fetch a... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | f5a890431069e39e7eb98c8c81da83629b10b51afe674db42bf829dbc0abba43 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/github-tools/github git https://github.com/github-tools/github.git |
| octolytics-dimension-user_id | 16116810 |
| octolytics-dimension-user_login | github-tools |
| octolytics-dimension-repository_id | 3640925 |
| octolytics-dimension-repository_nwo | github-tools/github |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 3640925 |
| octolytics-dimension-repository_network_root_nwo | github-tools/github |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 26c781f26acd529068611189d77d95c57099561a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width