[AI Gateway] Move worker binding methods to Using AI Gateway section#31403
[AI Gateway] Move worker binding methods to Using AI Gateway section#31403ethulia wants to merge 2 commits into
Conversation
ethulia
requested review from
a team,
aninibread,
kathayl,
kflansburg,
mattrothenberg,
mchenco,
palashgo,
roerohan,
superhighfives and
zeke
as code owners
Review✅ No style-guide issues found in commit CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
github-actions
Bot
added
product:ai-gateway
|
This PR requires additional review attention because it affects the following areas: RedirectsThis PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:
|
github-actions
Bot
assigned
aninibread,
kathayl,
kflansburg,
mattrothenberg,
mchenco,
palashgo,
roerohan,
superhighfives,
thebongy and
zeke
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
![ask-bonk[bot]](https://avatars.githubusercontent.com/in/2454580?s=60&v=4){kind=small}
There was a problem hiding this comment.
Summary
This PR cleanly moves the Workers Bindings reference from the Integrations section to the Using AI Gateway section. All internal links are updated and a redirect is in place — good attention to avoiding broken links.
Issues to address
1. Sidebar order collision in usage/ directory ⚠️
src/content/docs/ai-gateway/usage/worker-binding-methods.mdx still has:
sidebar:
order: 1However, src/content/docs/ai-gateway/usage/rest-api.mdx also has sidebar.order: 1. Having duplicate orders in the same directory produces ambiguous sidebar ordering in Starlight.
Suggestion: Update the moved file to a unique order. The existing files in usage/ currently use orders 1, 2, and 3 (index), so setting the moved file to order: 4 (or reordering the section intentionally) would resolve the collision.
2. Empty PR description 📝
The PR body contains only the unchecked template. Please add a short description explaining the rationale for the move (for example, why this content fits better under Usage than Integrations).
3. Changelog
No new changelog entry is required for a pure reorganization, but make sure the checklist in the PR description is intentionally filled out rather than left blank.
Labels
product:ai-gateway— appropriatesize/s— accurateproduct:changelog— accurate because a historical changelog link is updated
No additional labels needed.
|
Posted a request-changes review on PR #31403. What the PR does Labels Issues flagged
|
|
/bonk review this |
There was a problem hiding this comment.
Summary
Code changes look correct. The redirect, internal link updates, and sidebar re-ordering are all in place.
One remaining issue
Empty PR description — The PR body is still the unchecked template. Please add a brief rationale for moving the Workers Bindings page from Integrations to Usage.
|
Posted a review on PR #31403. What I checked
What remains
|
|
/bonk review this |
There was a problem hiding this comment.
Summary
Code changes are correct. The file move, redirect, internal link updates, and sidebar re-ordering are all properly handled.
One remaining issue
PR description — The body is still the unchecked template. Please add a brief rationale for moving the Workers Bindings page from Integrations to Usage (for example, why this content fits better under Usage). This was previously flagged and remains unaddressed.
|
Posted a review on PR #31403. What I checked
What I flagged
|
12 participants
Summary
Move worker binding methods to Using AI Gateway section
Screenshots (optional)
Documentation checklist