Skip to content
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

Fix the missed find_checked_ids_with_rbac #951

Merged

Conversation

gberginc
Copy link
Contributor

@gberginc gberginc commented Apr 6, 2017

It appears that one occurrence of the previously used find_checked_items_with_rbac was missed while renaming the function. This one is used while deleting cloud volumes.

@miq-bot add_label bug

It appears that one occurrence of the previously used
`find_checked_items_with_rbac` was missed while renaming the function.
This one is used while deleting cloud volumes.
@miq-bot miq-bot added the bug label Apr 6, 2017
@miq-bot
Copy link
Member

miq-bot commented Apr 6, 2017

Checked commit gberginc@f921685 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🍪

@martinpovolny martinpovolny merged commit 6cf6191 into ManageIQ:master Apr 7, 2017
@martinpovolny martinpovolny added this to the Sprint 58 Ending Apr 10, 2017 milestone Apr 7, 2017
simaishi pushed a commit that referenced this pull request Apr 7, 2017
Fix the missed `find_checked_ids_with_rbac`
(cherry picked from commit 6cf6191)
@simaishi
Copy link
Contributor

simaishi commented Apr 7, 2017

Fine backport details:

$ git log -1
commit 9fb9b8710ca7f95e1726ed4c64052fbdeff434df
Author: Martin Povolny <[email protected]>
Date:   Fri Apr 7 15:56:34 2017 +0200

    Merge pull request #951 from gberginc/fix/find_checked_ids_with_rbac
    
    Fix the missed `find_checked_ids_with_rbac`
    (cherry picked from commit 6cf619114738f873a6095297d8925c942bf73d0a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants