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 REST API docs to include default and max values for page_size #3210

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vishwamartur
Copy link

Related to #3135

Update REST APIs documentation to include default and max values for page_size flag

  • Client Code Changes

    • Update getStoreMessagesV3 function in waku/waku_api/rest/store/client.nim to set the default value of page_size to 20.
  • Handler Code Changes

    • Update installStoreApiHandlers procedure in waku/waku_api/rest/store/handlers.nim to enforce the default value of page_size to 20.
    • Enforce the max value of page_size to 100 in installStoreApiHandlers procedure.
  • Documentation Changes

    • Add a note in docs/api/rest-api.md mentioning the default value of page_size is 20 and the max value is 100.
    • Add a note in docs/operators/how-to/configure-rest-api.md mentioning the default value of page_size is 20 and the max value is 100.

Related to waku-org#3135

Update REST APIs documentation to include default and max values for `page_size` flag

* **Client Code Changes**
  - Update `getStoreMessagesV3` function in `waku/waku_api/rest/store/client.nim` to set the default value of `page_size` to 20.

* **Handler Code Changes**
  - Update `installStoreApiHandlers` procedure in `waku/waku_api/rest/store/handlers.nim` to enforce the default value of `page_size` to 20.
  - Enforce the max value of `page_size` to 100 in `installStoreApiHandlers` procedure.

* **Documentation Changes**
  - Add a note in `docs/api/rest-api.md` mentioning the default value of `page_size` is 20 and the max value is 100.
  - Add a note in `docs/operators/how-to/configure-rest-api.md` mentioning the default value of `page_size` is 20 and the max value is 100.
Copy link

github-actions bot commented Dec 14, 2024

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "Update REST API docs to include default and max values for page_size". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:

  • chore
  • docs
  • feat
  • fix
  • refactor
  • style
  • test

…ult and max values for `page_size`

* Set default value of `page_size` to 20 and max value to 100 in `getStoreMessagesV3` function in `waku/waku_api/rest/store/client.nim`
* Enforce default value of `page_size` to 20 and max value to 100 in `installStoreApiHandlers` procedure in `waku/waku_api/rest/store/handlers.nim`
* Add notes in `docs/api/rest-api.md` and `docs/operators/how-to/configure-rest-api.md` mentioning the default and max values for `page_size`
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.

1 participant