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

Request members should allow access to users with admin role #15163

Merged

Conversation

imtayadeway
Copy link
Contributor

This should have been done as part of
#15151, which fixed the
collection routes. This completes that work by fixing the member
routes.

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

Oops! Don't know how I missed this

@miq-bot add-label api, bug
@miq-bot assign @abellotti

This should have been done as part of
ManageIQ#15151, which fixed the
collection routes. This completes that work by fixing the member
routes.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1421878
@miq-bot
Copy link
Member

miq-bot commented May 19, 2017

Checked commit imtayadeway@c720cc3 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
4 files checked, 0 offenses detected
Everything looks fine. 🍪

@abellotti
Copy link
Member

LGTM!! Thanks @imtayadeway 🎵

@abellotti abellotti merged commit 1cc4ea0 into ManageIQ:master May 22, 2017
@abellotti abellotti added this to the Sprint 61 Ending May 22, 2017 milestone May 22, 2017
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