-
Notifications
You must be signed in to change notification settings - Fork 146
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
Enable admins to see workspace details and operations without a workspace role #3722
Conversation
…he error message. Fixes microsoft#2363
/test-force-approve UI only |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 1d16c94) (in response to this comment from @marrobi) |
Pull request was converted to draft
3524865
to
ad6676f
Compare
ad6676f
to
40aae26
Compare
/test-force-approve UI only |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 02bb546) (in response to this comment from @marrobi) |
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 good, just some minor suggestions
Co-authored-by: James Griffin <me@JamesGriff.in>
/test-force-approve UI only |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit eb2adc4) (in response to this comment from @marrobi) |
/test-force-approve UI only |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit be914b9) (in response to this comment from @marrobi) |
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
/test-force-approve UI only |
🤖 pr-bot 🤖 ✅ Marking tests as complete (for commit 5ad3dd0) (in response to this comment from @marrobi) |
Fixes #2363
How is this addressed