Skip to content

Updates for EmbeddedAnsible Workflows Provisioning#9921

Open
agrare wants to merge 1 commit into
ManageIQ:masterfrom
agrare:embedded_ansible_provision_dialogs
Open

Updates for EmbeddedAnsible Workflows Provisioning#9921
agrare wants to merge 1 commit into
ManageIQ:masterfrom
agrare:embedded_ansible_provision_dialogs

Conversation

@agrare
Copy link
Copy Markdown
Member

@agrare agrare commented Mar 24, 2026

Required for:

image

TODO:

  • Credentials dropdowns pre-populate if there is a single value and that can't be cleared

@agrare agrare requested a review from a team as a code owner March 24, 2026 17:22
@agrare agrare force-pushed the embedded_ansible_provision_dialogs branch from ae1e9dd to bbf3984 Compare March 26, 2026 17:12
@Fryguy
Copy link
Copy Markdown
Member

Fryguy commented Mar 26, 2026

As I don't see the change here in this PR, do you know why the tab has a lower-case-p "provision" ?

@Fryguy Fryguy self-assigned this Mar 26, 2026
@agrare
Copy link
Copy Markdown
Member Author

agrare commented Mar 26, 2026

As I don't see the change here in this PR, do you know why the tab has a lower-case-p "provision" ?

Ha that's a great question that I've been asking as well :) I have no idea yet. The provision dialog has :description: Provision [ref] so I'm still trying to figure that out.

@agrare agrare force-pushed the embedded_ansible_provision_dialogs branch 2 times, most recently from bed85ec to 08e69ed Compare March 27, 2026 16:25
@agrare agrare force-pushed the embedded_ansible_provision_dialogs branch from 08e69ed to 6d8001c Compare May 21, 2026 18:57
@miq-bot
Copy link
Copy Markdown
Member

miq-bot commented May 21, 2026

Checked commit agrare@6d8001c with ruby 3.3.10, rubocop 1.86.0, haml-lint 0.73.0, and yamllint 1.37.1
2 files checked, 2 offenses detected

app/views/miq_request/_prov_configuration_script_dialog.html.haml

  • ⚠️ - Line 43 - Line is too long. [139/80]

app/views/miq_request/_prov_field.html.haml

  • ⚠️ - Line 292 - Line is too long. [82/80]

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