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

Update API documentation to mention pagination #11115

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

lyrandy
Copy link
Contributor

@lyrandy lyrandy commented Sep 21, 2024

What does this PR do?

This PR updates API documentation to mention pagination, and how to control the pagination.

Motivation

I had a Traefik deployment that had over 100 HTTP routers, and was perplexed as to why some were missing in my API queries. I eventually came across #6917 , and thought documentation updates would be helpful to others.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

I built the documentation locally according to Method 1, and looked at it on my local workstation to see if my change looked ok. I then used make docs to clean and verify the docs.

...
=== Checking HTML content...
= Documentation checked successfully.

Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

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

Thanks!

@kevinpollet kevinpollet force-pushed the feature/docs_api_pagination_6917 branch from 63764af to c0bfc03 Compare September 27, 2024 12:50
@kevinpollet kevinpollet changed the title Update API documentation to mention pagination. Update API documentation to mention pagination Sep 27, 2024
lyrandy and others added 2 commits September 27, 2024 14:54
Pagination can be controlled using the `page` and `per_page` HTTP Query
parameters.
@kevinpollet kevinpollet force-pushed the feature/docs_api_pagination_6917 branch from c0bfc03 to 3737155 Compare September 27, 2024 12:55
Copy link
Member

@kevinpollet kevinpollet left a comment

Choose a reason for hiding this comment

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

Thanks 👍

@traefiker traefiker merged commit e485edb into traefik:v2.11 Sep 27, 2024
4 checks passed
@lyrandy
Copy link
Contributor Author

lyrandy commented Sep 27, 2024

Thanks all! Happy to contribute to Traefik!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants