docs(admin): freshness pass on organization members page - #25669
Conversation
Refresh members.md for Docker Home: tab invite methods, async CSV export email, remove-from-org steps, and clearer invitation/seat guidance. Retarget the SSO FAQ link to the org-removal section. Co-authored-by: Cursor <cursoragent@cursor.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
Clean freshness pass — prose tightened, AI-isms removed, UI terminology corrected, and the new tab layout improves scannability. The SSO FAQ anchor is updated consistently. No redirects are needed (page URL unchanged). One minor nit below.
| - Member of Organizations: Organizations the user belongs to within the | ||
| company | ||
| - Invited to Organizations: Organizations the user is invited to within the | ||
| company No newline at end of file |
There was a problem hiding this comment.
[LOW] Missing newline at end of file
The file ends without a trailing newline (\n). This doesn't affect rendered output on Docker Docs, but it violates POSIX text-file conventions and can create noisy diffs in future edits. Adding a single blank line after the last bullet would fix this.
docker-agent
left a comment
docker-agent
left a comment
There was a problem hiding this comment.
Assessment: 🟡 NEEDS ATTENTION
Freshness pass is well-executed — cleaner prose, consistent terminology (Docker Home, Actions menu), and the new tabs layout and "Remove a member from the organization" section are good additions. Three minor issues flagged inline.
|
|
||
| ## Export members CSV file | ||
|
|
||
| {{< summary-bar feature_name="Admin orgs" >}} |
There was a problem hiding this comment.
[MEDIUM] summary-bar shortcode removed — "Administrators only" constraint may be lost
The {{< summary-bar feature_name="Admin orgs" >}} shortcode rendered two metadata fields (from data/summary.yaml):
- Subscription: Team or Business
- For: Administrators
The replacement prose — "Owners of Docker Team or Business organizations can export a CSV file" — captures the subscription tier but doesn't convey the Administrators-only scope. If this restriction still applies, consider adding a note such as: "Only organization owners can export the member list."
| ## Export members CSV file | ||
|
|
||
| {{< summary-bar feature_name="Admin orgs" >}} | ||
| Owners of Docker Team or Business organizations can export a CSV file of all members. Docker generates the file |
There was a problem hiding this comment.
[LOW] Long unwrapped prose line (111 chars)
This line is 111 characters — longer than the ~80-char soft wrap used throughout the rest of the file. The sentence continues on the next line, making the paragraph read as a run-on in raw Markdown. Consider wrapping:
Owners of Docker Team or Business organizations can export a CSV file of all
members. Docker generates the file asynchronously and emails it to the owner
when it's ready.| 1. Select the **download** icon to export a CSV file of all members. | ||
| - Member of Organizations: Organizations the user belongs to within the | ||
| company | ||
| - Invited to Organizations: Organizations the user is invited to within the |
There was a problem hiding this comment.
[LOW] Missing newline at end of file
The diff shows \ No newline at end of file — members.md no longer ends with a trailing newline. This can cause noisy diffs when editors auto-add it. Add a blank line after the last bullet.
| ## Export members CSV file | ||
|
|
||
| {{< summary-bar feature_name="Admin orgs" >}} | ||
| Owners of Docker Team or Business organizations can export a CSV file of all members. Docker generates the file |
There was a problem hiding this comment.
We don't restrict it to Team or Business, any org can export
3 participants
Updates to organization members page to reflect async csv exports. Need to confirm a couple things:
Move company csv call out into company docs...