-
Notifications
You must be signed in to change notification settings - Fork 358
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 double ng-if #6382
Fix double ng-if #6382
Conversation
@tinaafitz @lfu Can one of you verify if "Hosts" field should be visible for Playbook methods? I think changes in #5295 were added to support playbook type automate methods so we should go with the field being visible. what do you think? |
|
Remove check "#{ng_model}.#{prefix}_inventory!==undefined" as it's always set for Ansible Playbook and "#{ng_model}.location==='playbook'" makes sure it's not shown for other methods/catalog items
aefc299
to
03bbb70
Compare
Checked commit ZitaNemeckova@03bbb70 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 |
@miq-bot remove_label wip @lfu Thanks :) Fixed accordingly. @h-kataria please have a look, thanks :) |
CI failure #4921 (comment) |
@miq-bot add_label wip, cleanup, ivanchuk/no
Fix following error in spec log:
Places to check it works:
Services -> Catalogs -> Catalog Items -> create/edit -> Hosts show only for Ansible Playbook
Automation -> Automate -> Explorer -> create/edit a method -> Hosts show only for Ansible Playbook
Hosts pic:
Remove check
"#{ng_model}.#{prefix}_inventory!==undefined"
as it's always set for Ansible Playbook and"#{ng_model}.location==='playbook'"
makes sure it's not shown for other methods/catalog items.