-
Notifications
You must be signed in to change notification settings - Fork 357
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
Adding methods for treeSelector into Dialog Editor's controller #2591
Conversation
d4752e0
to
0269ce9
Compare
0269ce9
to
7eaba50
Compare
@skateman @karelhala @martinpovolny please review |
Checked commits romanblanco/manageiq-ui-classic@d66b9d8~...4b7626a with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 app/controllers/tree_controller.rb
|
Restarted travis. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, after release we have to change the bindings to something more meaningful, this is getting too complex.
@martinpovolny looks good, can you merge it please? Alongside with this we need to merge ManageIQ/ui-components#198 and release new version of |
This needs a new version of ui-component. Will wait to backport to gaprindashvili until new version becomes ready. Marking as conflict for now. cc @himdel |
@romanblanco I've just released manageiq-ui-components 1.0.1. Can you please follow the rest of the process? |
Adding methods for treeSelector into Dialog Editor's controller (cherry picked from commit 7b19340) https://bugzilla.redhat.com/show_bug.cgi?id=1510244
Gaprindashvili backport details:
|
Need to be merged together with ManageIQ/ui-components#198
Adding methods for
treeSelector
component that is necessary for selecting automate methods in dialog editorLinks
Steps for Testing/QA
Automate -> Automation -> Customization -> Service Dialogs -> Add / Edit Dialog