Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Org User status to roles description and table #75

Merged
merged 6 commits into from
Sep 15, 2017
Merged

Add Org User status to roles description and table #75

merged 6 commits into from
Sep 15, 2017

Conversation

brittag
Copy link
Contributor

@brittag brittag commented Aug 23, 2017

In order for my end-users in our regulated environment to have confidence that they are managing access to their orgs appropriately according to their access control requirements, they need to understand CF org user management in depth, including hidden/implied roles.

This is a proposal for documenting Org User status, following up on my comment at cloudfoundry/cli#781 (comment) and this discussion in #cf-docs in the Cloud Foundry Slack. I figure an illustration of what might be helpful to us can be helpful. :)

This is a best guess at what the documentation should look like - I haven't verified all of the checkboxes here, and I haven't tried running this locally to check for formatting/display errors. I verified that if I have org user status (but no other roles) on an org, I can "View users and roles".

@cfdreddbot
Copy link

Hey brittag!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/150491632

The labels on this github issue will be updated when the story is started.

@seviet
Copy link
Contributor

seviet commented Aug 23, 2017

@zrob Hey Zach-- could you review the addition of this column to the roles table and let us know if there should be any changes? Many thanks!!

@brittag
Copy link
Contributor Author

brittag commented Sep 9, 2017

Thanks @seviet! @zrob Here are screenshots of the changes, in case that's helpful for reviewing this (I got it set up locally now):

screen shot 2017-09-08 at 5 24 34 pm

screen shot 2017-09-08 at 5 25 08 pm

@zrob
Copy link
Contributor

zrob commented Sep 15, 2017

LGTM

@seviet seviet merged commit b19f9ca into cloudfoundry:master Sep 15, 2017
@seviet
Copy link
Contributor

seviet commented Sep 15, 2017

Thanks @brittag and @zrob!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants