-
Notifications
You must be signed in to change notification settings - Fork 897
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
Apply RBAC filtering to the foreman tree objects #5665
Conversation
@@ -946,6 +944,21 @@ def process_show_list(options = {}) | |||
super | |||
end | |||
|
|||
def find_record(db, id) |
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.
suggest renaming db
to klass
or model
@gtanzillo Please review |
ba8eb58
to
f0ef808
Compare
👍 Looks good to me |
f0ef808
to
a1a5ad1
Compare
Checked commits AparnaKarve/manageiq@6ba3d19~...a1a5ad1 with ruby 2.2.3, rubocop 0.34.2, and haml-lint 0.13.0 app/controllers/application_controller.rb
app/controllers/provider_foreman_controller.rb
app/presenters/tree_builder.rb
app/presenters/tree_builder_foreman.rb
|
Apply RBAC filtering to the foreman tree objects
BZ1247715 - Apply RBAC filtering to the foreman tree objects Clean cherry-pick Upstream PR - ManageIQ#5665 Apply RBAC filtering to the Foreman tree nodes based on the logged-in user's tag settings. https://bugzilla.redhat.com/show_bug.cgi?id=1247715 See merge request !582
Apply RBAC filtering to the Foreman tree nodes based on the logged-in user's tag settings.
https://bugzilla.redhat.com/show_bug.cgi?id=1247715