From 4cfa4eaf8e1ef7c52dc4e94d4144d63377693f72 Mon Sep 17 00:00:00 2001 From: nixocio Date: Fri, 29 Oct 2021 16:14:23 -0400 Subject: [PATCH] Update validators for Misc Auth Edit * Update SharedFields to use number validator instead of integer * Use number validation for SESSIONS_PER_USER See: https://github.com/ansible/tower/issues/5396 --- awx/ui/src/screens/Setting/shared/SharedFields.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/awx/ui/src/screens/Setting/shared/SharedFields.js b/awx/ui/src/screens/Setting/shared/SharedFields.js index d085761a93..d273737f50 100644 --- a/awx/ui/src/screens/Setting/shared/SharedFields.js +++ b/awx/ui/src/screens/Setting/shared/SharedFields.js @@ -22,7 +22,7 @@ import CodeEditor from 'components/CodeEditor'; import { PasswordInput } from 'components/FormField'; import { FormFullWidthLayout } from 'components/FormLayout'; import Popover from 'components/Popover'; -import { combine, integer, minMaxValue, required, url } from 'util/validators'; +import { combine, minMaxValue, required, url, number } from 'util/validators'; import AlertModal from 'components/AlertModal'; import RevertButton from './RevertButton'; @@ -365,12 +365,11 @@ const InputField = ({ name, config, type = 'text', isRequired = false }) => { const validators = [ ...(isRequired ? [required(null)] : []), ...(type === 'url' ? [url()] : []), - ...(type === 'number' - ? [integer(), minMaxValue(min_value, max_value)] - : []), + ...(type === 'number' ? [number(), minMaxValue(min_value, max_value)] : []), ]; const [field, meta] = useField({ name, validate: combine(validators) }); const isValid = !(meta.touched && meta.error); + return config ? ( { validated={isValid ? 'default' : 'error'} >