-
Notifications
You must be signed in to change notification settings - Fork 55
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
DXCDT-313: Add ability to view a user's assigned roles #604
Conversation
internal/cli/users_roles.go
Outdated
return fmt.Errorf("failed to find roles for user with ID %s: %w", inputs.ID, err) | ||
} | ||
|
||
cli.renderer.RoleList(userRoleList.Roles) |
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.
Currently this none-found hint is displayed: "Use 'auth0 roles create' to add one"
Eventually this help text should display something regarding the assignment of a role. Ex: "Use auth0 users roles assign
"
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.
Also: "No roles available." should read something like "No roles assigned to user".
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 now displays in 3094faf
(#604)
=== tenant.eu.auth0.com roles assigned to user (0)
No roles assigned to user available.
▸ Use 'auth0 users roles assign' to add one
ac216d8
to
3094faf
Compare
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.
Given that this is a new command, can we add some type of commander test?
fb39943
to
fdc6df7
Compare
Update user roles number flat help text
73a6451
to
aaf0137
Compare
🔧 Changes
Adding the ability to view a user's assigned roles.
📚 References
🔬 Testing
📝 Checklist