Skip to content

Add limit and cursor query parameters for Workers for Platforms#28765

Open
danielErazoUrrego-cmd wants to merge 2 commits intocloudflare:productionfrom
danielErazoUrrego-cmd:patch-8
Open

Add limit and cursor query parameters for Workers for Platforms#28765
danielErazoUrrego-cmd wants to merge 2 commits intocloudflare:productionfrom
danielErazoUrrego-cmd:patch-8

Conversation

@danielErazoUrrego-cmd
Copy link
Contributor

Adding documentation for limit and cursor supported query parameters on the list scripts endpoint for Workers for Platforms

Adding documentation for limit and cursor supported query parameters on the list scripts endpoint for Workers for Platforms
@elithrar
Copy link
Collaborator

elithrar commented Mar 4, 2026

/bonk review this for accuracy and content.

Copy link
Contributor

@ask-bonk ask-bonk bot left a comment

Choose a reason for hiding this comment

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

Two issues to fix before merge: inconsistent indentation (spaces vs tabs) and the second APIRequest example should show both limit and cursor together for a more useful pagination example. Posted inline suggestions.

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 4, 2026

Review posted. Two issues flagged:

  1. Indentation mismatch — the new APIRequest block uses 4 spaces while the rest of the file uses tabs.
  2. Redundant example — the second APIRequest shows only limit, which duplicates the endpoint from the block above without demonstrating much new. Suggested showing both limit and cursor together so readers see the complete pagination pattern.

github run

Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
@irvinebroque
Copy link
Contributor

/bonk please address "Redundant example — the second APIRequest shows only limit, which duplicates the endpoint from the block above without demonstrating much new. Suggested showing both limit and cursor together so readers see the complete pagination pattern."

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 4, 2026

APIError: This model does not support assistant message prefill. The conversation must end with a user message.

github run

@ask-bonk
Copy link
Contributor

ask-bonk bot commented Mar 4, 2026

@irvinebroque Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

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.

4 participants