Skip to content

sbx: update labels#25141

Merged
craig-osterhout merged 1 commit into
docker:mainfrom
craig-osterhout:sbx-update
May 21, 2026
Merged

sbx: update labels#25141
craig-osterhout merged 1 commit into
docker:mainfrom
craig-osterhout:sbx-update

Conversation

@craig-osterhout
Copy link
Copy Markdown
Contributor

Description

Update labels for sandbox

Related issues or tickets

https://docker.slack.com/archives/C04300R4G5U/p1779390142616439

Reviews

  • Editorial review
  • Product review

@craig-osterhout craig-osterhout requested a review from dvdksn as a code owner May 21, 2026 19:26
@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit c58d7c7
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a0f5d09cf80690007370267
😎 Deploy Preview https://deploy-preview-25141--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR intentionally updates the Early Access labeling for Docker Sandboxes documentation:

  • Removes the Early Access sidebar badge and summary-bar shortcode from the root Sandboxes section and all non-customize pages (indicating those features have reached GA)
  • Retains the Early Access sidebar badge and summary-bar on the customize/ sub-section pages (customize/_index.md, kits.md, templates.md), where new functionality is still in Early Access

The selective labeling is a coherent editorial decision. Pages without the summary-bar shortcode still communicate subscription requirements through inline prose ("Available on a separate paid subscription"). No style violations, markdown errors, AI-isms, or missing redirects were found.

@craig-osterhout craig-osterhout requested a review from a team May 21, 2026 19:34
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label May 21, 2026
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟡 NEEDS ATTENTION

This PR reorganizes Early Access labeling across Docker Sandboxes docs — removing summary-bar shortcodes from most pages and consolidating Early Access badges to the customize/ sub-section. The intent is clear, but the execution leaves two labeling inconsistencies worth addressing.

Comment thread content/manuals/ai/sandboxes/customize/_index.md
Comment thread content/manuals/ai/sandboxes/_index.md
Copy link
Copy Markdown
Contributor

@akristen akristen left a comment

Choose a reason for hiding this comment

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

Did a side-by-side comparison, confirmed I understood/saw the correct pattern, everything looks good!

@craig-osterhout craig-osterhout merged commit adb9861 into docker:main May 21, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ai status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants