Skip to content
This repository was archived by the owner on Aug 18, 2025. It is now read-only.

Conversation

khorne3
Copy link
Contributor

@khorne3 khorne3 commented Nov 5, 2021

I played with a bunch of different charts, including this one, but in the end, I didn't think the added complexity would be helpful to anyone reading our user perms doc. So, I just added a brief summary of how the permissions are additive (mostly).

Will also add the automated tables when we have that ready.

@khorne3 khorne3 requested a review from jawnsy November 5, 2021 18:57
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #17620: User perms table difficult to read.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2021

✨ Coder.com for PR #698 deployed! It will be updated on every commit.

Copy link
Contributor

@jawnsy jawnsy left a comment

Choose a reason for hiding this comment

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

LGTM!

Katie Horne and others added 4 commits November 8, 2021 08:06
@khorne3 khorne3 merged commit eb54211 into main Nov 8, 2021
@khorne3 khorne3 deleted the katiehorne/sc-17620 branch November 8, 2021 14:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants