[Version Management] Update API limitation, fix troubleshooting page, add code owner#31804
Conversation
|
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.
Review summary
Changes
- CODEOWNERS: Adds
@jhutchings1 @cloudflare/product-ownersfor/src/content/partials/version-management/. This is consistent with the existing docs path and is a sensible addition. - Product limitations partial: Updates the Cloudflare API section to reflect that environments can now be managed via the public Environments API, while maintaining that version snapshot operations remain dashboard-only.
Verification
- ✅ The internal link
/api/resources/zones/subresources/environments/resolves correctly on the live site. - ✅ Frontmatter and component import are correctly structured for a partial.
- ✅ PR title follows the
[Product] descriptionconvention.
Flags
- Inline comment posted on a subject-verb agreement issue in the new dashboard-only bullet ("is" → "are").
- The Build check is still pending; assuming it passes, this change is straightforward.
Labels
size/xs is already applied and appropriate.
|
Review completed for PR #31804 Summary of changes
Labels applied
Issues flagged
Verification
I posted a review comment on the PR detailing these findings. Note: I was unable to post an inline review comment via the available GitHub API in this environment, so the grammar flag is included in the main review body instead. |
|
Preview URL: https://dcf481df.preview.developers.cloudflare.com Files with changes (up to 15) |
- Document that Version Management environments can be managed through the public Environments API. Only the creation, cloning, and editing of zone versions themselves remain dashboard-only. - Add @jhutchings1 as code owner for /src/content/partials/version-management/. Replaces PR #31525 (originally opened by external contributor @Techsurfer1981, SFSC case 02109410).
|
@Techsurfer1981 here is the new PR |
- Remove the claim that Version Management does not expose a public API: the Environments API is now public. - Remove the '10001 authentication error is expected behavior' paragraph: no longer accurate now that the API is available. - Keep the Terraform mutual-exclusion guidance under a renamed section.
ngayerie
force-pushed
the
ngayerie/version-management-api-limitation
branch
from
ccb451d to
dcf481d
Compare
ngayerie
changed the title
Review💡 1 suggestion found in commit Code ReviewThis code review is in beta and may not always be helpful — use your judgment. No code review issues found. ConventionsChecks PR title, description, and redirect checklist. No convention issues found. Style Guide ReviewSuggestions (1)
RedirectsNo missing redirect entries found. CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
Hi @jhutchings1 , could you please approve this PR? |
3 participants
Summary
Update the Version Management documentation to reflect that the public Environments API is available, and add @jhutchings1 as code owner for the version-management partials.
Changes
partials/version-management/product-limitations.mdxdocs/version-management/troubleshooting.mdx10001error is expected behavior; keep the Terraform mutual-exclusion guidance under a renamed section.github/CODEOWNERS@jhutchings1as code owner for/src/content/partials/version-management/Replaces #31525.