Skip to content

clarify roles permissions#24813

Merged
akristen merged 1 commit intodocker:mainfrom
akristen:repo-permissions
Apr 17, 2026
Merged

clarify roles permissions#24813
akristen merged 1 commit intodocker:mainfrom
akristen:repo-permissions

Conversation

@akristen
Copy link
Copy Markdown
Contributor

Docs request from channel

@akristen akristen self-assigned this Apr 17, 2026
@akristen akristen added the area/admin Relates to Docker Admin label Apr 17, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 874439a
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69e291ab447a3d0008ec2cca
😎 Deploy Preview https://deploy-preview-24813--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.

Copy link
Copy Markdown
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

LGTM.
One suggestion for existing content that's not necessary.


When you add members to teams, you can grant additional repository permissions
You can grant repository permissions to members
beyond their organization role:
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.

For some reason, my brain short-circuited on this part and it took me a while to understand what it was trying to convey. I'm guessing it's just saying there are 2 different permission checks. Maybe provide an example or some flow diagram of permission checks for different combinations of each.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm going to leave this as is but I've added it to a to do list for when I do a freshness sweep on this part of the admin docs. It's a good callout

@akristen akristen merged commit d131957 into docker:main Apr 17, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants