-
Notifications
You must be signed in to change notification settings - Fork 27
organization users #712
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
organization users #712
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that the roles need to come before the invitation part. If you don't understand the roles, how can you be expected to assign a role to a new user?
== Organization-level roles | ||
|
||
The following roles are available at the org level and these are assigned via invitation: | ||
Roles at the organization level determine what administrative capabilities a user has across all projects within the organization. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to move back up to the top.
Co-authored-by: Jessica Wright <49636617+AlexicaWright@users.noreply.github.com>
Co-authored-by: Jessica Wright <49636617+AlexicaWright@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we're getting there!
Co-authored-by: Jessica Wright <49636617+AlexicaWright@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't the original task to clarify that you need to invite someone on the project-level but that you can later edit their automatically-assigned org role? I think we should put the admonition back towards the top, or somewhere visible to make this clear. At least until they fix this. Apart from that, looks great! Well done!
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thank you @fiquick !
No description provided.