From 8227054d116dd58d715447ac8cd64daed1503308 Mon Sep 17 00:00:00 2001 From: Alex Corey Date: Tue, 4 Feb 2020 16:30:06 -0500 Subject: [PATCH] Adds WorkflowJobTemplate Add and Edit form and functionality. Fields work and forms save Adds Webhook form fields and tooltip to VariablesField component Alpha order Fixes contentLoading issue in PR and enables Launch on JT List Adds page section to fix render issue Adds subform restore functionality and addresses PR issues Adds pageSection to jt add form and fixes other PR issues -Fixes spelling error on WFJTDetail -Adds page section to JT Add Form to fix styling issue -Adds spacing between functions -Fixes form submission error by allowing state to handle the lookups while formik only handles their ids. Fixes styling issues, navigation, props, and adds useRequest hook -Add functionality to remove chips from look up fields -Removes uncessary custome styling from -Removes uncessary Form Group wrappers -Adds internationalization to webhook key string. -Adds field level error handling -updates tests -Adds initial null value to form submit error Adds error handling to submit labels, prevents uncessary api call The unecessary api call is for the webhook credential id. If there is no webhook service we do not want the api to make a call for get the webhook credential type id. Adds error handling test to add and edit form. Updates Form component Updates tests to a real error. Fixes extra vars bug --- .../src/screens/Template/shared/WorkflowJobTemplateForm.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/awx/ui_next/src/screens/Template/shared/WorkflowJobTemplateForm.jsx b/awx/ui_next/src/screens/Template/shared/WorkflowJobTemplateForm.jsx index 323010ce34..770f665982 100644 --- a/awx/ui_next/src/screens/Template/shared/WorkflowJobTemplateForm.jsx +++ b/awx/ui_next/src/screens/Template/shared/WorkflowJobTemplateForm.jsx @@ -206,7 +206,7 @@ function WorkflowJobTemplateForm({ inventory: template?.summary_fields?.inventory?.id || null, organization: template?.summary_fields?.organization?.id || null, labels: template.summary_fields?.labels?.results || [], - extra_vars: template.variables || '---', + extra_vars: template.extra_vars || '---', limit: template.limit || '', scm_branch: template.scm_branch || '', allow_simultaneous: template.allow_simultaneous || false,