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

Fixed class name and key name in add/edit of user role code #11999

Merged
merged 1 commit into from Oct 18, 2016

Conversation

h-kataria
Copy link
Contributor

Fixed inssue introduced in sha: 93f3f5e in #11466

@martinpovolny @romanblanco please review. This fixes an issue introduced in #11466 that is preventing from adding User Roles in UI.

@h-kataria h-kataria changed the title Fixed class name and key name while to fix add/edit of user role Fixed class name and key name while adding fix add/edit of user role Oct 17, 2016
@h-kataria h-kataria changed the title Fixed class name and key name while adding fix add/edit of user role Fixed class name and key name in add/edit of user role code Oct 17, 2016
@miq-bot
Copy link
Member

miq-bot commented Oct 17, 2016

Checked commit h-kataria@cdc9419 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 👍

@romanblanco
Copy link
Member

@martinpovolny Tested in UI, fixes the issue 👍. Thank you, @h-kataria 😊

@martinpovolny martinpovolny merged commit 17bc43f into ManageIQ:master Oct 18, 2016
@martinpovolny martinpovolny added this to the Sprint 48 Ending Oct 24, 2016 milestone Oct 18, 2016
@h-kataria h-kataria deleted the user_role_add_fix branch October 28, 2016 20:40
@h-kataria
Copy link
Contributor Author

Need to backport to EUWE to fix https://bugzilla.redhat.com/show_bug.cgi?id=1390969

cc @chessbyte

chessbyte pushed a commit that referenced this pull request Nov 7, 2016
Fixed class name and key name in add/edit of user role code
(cherry picked from commit 17bc43f)

https://bugzilla.redhat.com/show_bug.cgi?id=1390969
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log -1
commit 02b4259290fc3b92799fbe611b0688c699eea583
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Tue Oct 18 09:57:16 2016 +0200

    Merge pull request #11999 from h-kataria/user_role_add_fix

    Fixed class name and key name in add/edit of user role code
    (cherry picked from commit 17bc43fcb6893b14737a5a63f7fddbdf404ed295)

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

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.

None yet

5 participants