-
Notifications
You must be signed in to change notification settings - Fork 8
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
Updated Workflows Credential Schema #30
Updated Workflows Credential Schema #30
Conversation
Checked commit DavidResende0@f0422ad with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint |
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.
LGTM - want @agrare to review.
{ | ||
:component => 'text-field', | ||
:label => N_('Name'), | ||
:helperText => N_('Name of this credential'), | ||
:name => 'name', | ||
:id => 'name' | ||
}, |
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.
If we're going to remove name from COMMON_ATTRIBUTES
then we need to add it to the list of allowed_params in .params_to_attributes
here (this is why the specs are failing)
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.
Adding name to the core set of allowed attributes in ManageIQ/manageiq#22570 fixes this.
Backported to
|
Updated Workflows Credential Schema (cherry picked from commit e49a229)
Removed the
Name
field from theManageIQ::Providers::Workflows::AutomationManager::Credential
type schema since name is a common field across credential types and should be in the front-end. Also added some validation to theReference
field since it is a required field.