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 MiqGroup#miq_user_role_name #17686

Merged
merged 1 commit into from
Jul 10, 2018
Merged

Conversation

kbrock
Copy link
Member

@kbrock kbrock commented Jul 10, 2018

virtual_delegate doesn't work too well with :through
manually unroll the delegate code

introduced in 83ad831

https://bugzilla.redhat.com/show_bug.cgi?id=1593171

@bdunne
Copy link
Member

bdunne commented Jul 10, 2018

Is it worth adding a test for this?

virtual_delegate doesn't work too well with :through
manually unroll the delegate code

introduced in 83ad831

https://bugzilla.redhat.com/show_bug.cgi?id=1593171
@kbrock kbrock force-pushed the group_role_name branch from 00a5f4d to 850df38 Compare July 10, 2018 15:34
@miq-bot
Copy link
Member

miq-bot commented Jul 10, 2018

Checked commit kbrock@850df38 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. ⭐

@bdunne bdunne merged commit e484d1a into ManageIQ:master Jul 10, 2018
@bdunne bdunne self-assigned this Jul 10, 2018
@bdunne bdunne added this to the Sprint 90 Ending Jul 16, 2018 milestone Jul 10, 2018
@kbrock kbrock deleted the group_role_name branch July 10, 2018 16:29
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.

3 participants