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
Restrict the Help->Documentation screen based on RBAC rules only #3728
Conversation
Checked commit skateman@9aa897b with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@skateman The BZ has cfme-5.8.z flag too.. Can this be |
@miq-bot add_label fine/yes |
Restrict the Help->Documentation screen based on RBAC rules only (cherry picked from commit 7e2fffe) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1569171
Fine backport details:
|
@skateman Can you take a look at Travis failure in File branch? I think that's coming from backport of this PR.... https://travis-ci.org/ManageIQ/manageiq-ui-classic/jobs/368272457 |
Restrict the Help->Documentation screen based on RBAC rules only (cherry picked from commit 7e2fffe) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1569170
Gaprindashvili backport details:
|
The documentation text is visible only for administrators even though the RBAC role can display the menu item for it. In this PR I'm dropping the
is_admin?
calls from the view and introducing an RBAC check in the controller.@miq-bot add_label bug, gaprindashvili/yes
@miq-bot add_reviewer @romanblanco
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1563241