-
Notifications
You must be signed in to change notification settings - Fork 356
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
Display checkable folders in the Alert Profile Assignment tree for ems_folder
#1747
Display checkable folders in the Alert Profile Assignment tree for ems_folder
#1747
Conversation
08da221
to
e9568cb
Compare
Also other checkbox related properties for Alert Profile Assignment will be set. This fixes Alert Profile assignment for ems_folders https://bugzilla.redhat.com/show_bug.cgi?id=1473321
e9568cb
to
d792052
Compare
d792052
to
7911572
Compare
Checked commits AparnaKarve/manageiq-ui-classic@9ee07bd~...7911572 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
LGTM, verified fix in UI. |
@AparnaKarve |
@AparnaKarve Backporting to Fine conflicts on both files. Can you please create a Fine PR? |
@simaishi Thanks. |
Backported to Fine via #1889 |
Backported to Euwe via ManageIQ/manageiq#15793 |
Fixed the Alert Profiles Assignment tree to display correct nodes when
Selected Folders
is selected and ensured that the folders are checkable.Before (Displays wrong nodes + uncheckable nodes)
![screen shot 2017-07-24 at 3 43 10 pm](https://user-images.githubusercontent.com/1538216/28551258-e132de72-709b-11e7-850d-f72e9ae4707b.png)
After (Displays correct nodes (folders) that are checkable)
![screen shot 2017-07-24 at 6 11 05 pm](https://user-images.githubusercontent.com/1538216/28551317-17b4866c-709c-11e7-88fd-0fa80de73402.png)
https://bugzilla.redhat.com/show_bug.cgi?id=1473321