Skip to content

SFS RN update - #25703

Open
aevesdocker wants to merge 1 commit into
docker:mainfrom
aevesdocker:sfs-updates
Open

SFS RN update#25703
aevesdocker wants to merge 1 commit into
docker:mainfrom
aevesdocker:sfs-updates

Conversation

@aevesdocker

Copy link
Copy Markdown
Contributor

Description

Removes RN item for feature that is behind a feature flag

Related issues or tickets

Reviews

  • Technical review
  • Editorial review
  • Product review

@netlify

netlify Bot commented Aug 4, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit eb4aa43
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a71ae12dcc4400008c2c13c
😎 Deploy Preview https://deploy-preview-25703--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.

@github-actions github-actions Bot added the area/desktop Issue affects a desktop edition of Docker. E.g docker for mac label Aug 4, 2026
@aevesdocker
aevesdocker requested a review from a team August 4, 2026 09:17

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The change cleanly removes a single release notes bullet for a feature still behind a feature flag. No markdown issues, no style violations, no redirect needed (only a list item was removed, not a page or section anchor). The diff is minimal and scope-preserving.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The change removes one release notes bullet for Synchronized file shares WSL2 backend support — correctly scoping the notes to exclude a feature that is still behind a feature flag. The removal is clean, no markdown issues, no redirects needed, and no scope drift.

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The PR removes a single release-notes list item for a feature that is behind a feature flag. The change is clean: no new lines are introduced, the surrounding Markdown list remains valid, and no page is removed or moved (so no redirects are needed). Nothing to flag.

@karman-docker

Copy link
Copy Markdown
Contributor

Hi @aevesdocker please hold on, there have been few discoveries https://docker.slack.com/archives/C0326G3P08M/p1785770552827119

While we will need docs update, the content of the update needs to be finalized. Tracking via https://docker.atlassian.net/browse/CSESC-1735

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/desktop Issue affects a desktop edition of Docker. E.g docker for mac

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants