diff --git a/frontend/components/forms/admin/AppConfigForm/AppConfigForm.tsx b/frontend/components/forms/admin/AppConfigForm/AppConfigForm.tsx index d6d914f838..e0dceb01ba 100644 --- a/frontend/components/forms/admin/AppConfigForm/AppConfigForm.tsx +++ b/frontend/components/forms/admin/AppConfigForm/AppConfigForm.tsx @@ -219,7 +219,7 @@ const AppConfigFormFunctional = ({ } if (enableHostExpiry) { - if (!hostExpiryWindow) { + if (!hostExpiryWindow || hostExpiryWindow <= 0) { errors.host_expiry_window = "Host expiry window must be a positive number"; } @@ -235,7 +235,7 @@ const AppConfigFormFunctional = ({ setFormErrors(errors); }; - // Validates forms when certain checkboxes and dropdowns are selected + // Validates forms when certain information is changed useEffect(() => { validateForm(); }, [ @@ -245,6 +245,7 @@ const AppConfigFormFunctional = ({ enableHostStatusWebhook, enableHostExpiry, agentOptions, + hostExpiryWindow, ]); // TOGGLE MODALS @@ -832,86 +833,73 @@ const AppConfigFormFunctional = ({

-
- If you need to specify a HELO domain,
you can do it here (Default: Blank)

' - } - /> -
-
- Turn this off (not recommended)
if you use a self-signed certificate
(Default: On)

' - } - > - Verify SSL certs -
-
-
- Detects if STARTTLS is enabled
in your SMTP server and starts
to use it. (Default: On)

' - } - > - Enable STARTTLS -
-
-
- When enabled, allows automatic cleanup
of hosts that have not communicated with Fleet
in some number of days. (Default: Off)

' - } - > - Host expiry -
-
-
- If a host has not communicated with Fleet in the specified number of days, it will be removed.

" - } - /> -
-
- When enabled, disables the ability to run live queries
(ad hoc queries executed via the UI or fleetctl). (Default: Off)

' - } - > - Disable live queries -
-
+ If you need to specify a HELO domain,
you can do it here (Default: Blank)

' + } + /> + Turn this off (not recommended)
if you use a self-signed certificate
(Default: On)

' + } + > + Verify SSL certs +
+ Detects if STARTTLS is enabled
in your SMTP server and starts
to use it. (Default: On)

' + } + > + Enable STARTTLS +
+ When enabled, allows automatic cleanup
of hosts that have not communicated with Fleet
in some number of days. (Default: Off)

' + } + > + Host expiry +
+ If a host has not communicated with Fleet in the specified number of days, it will be removed.

" + } + /> + When enabled, disables the ability to run live queries
(ad hoc queries executed via the UI or fleetctl). (Default: Off)

' + } + > + Disable live queries +
diff --git a/frontend/components/forms/admin/AppConfigForm/_styles.scss b/frontend/components/forms/admin/AppConfigForm/_styles.scss index d77e88113e..3c17108438 100644 --- a/frontend/components/forms/admin/AppConfigForm/_styles.scss +++ b/frontend/components/forms/admin/AppConfigForm/_styles.scss @@ -224,51 +224,5 @@ &__advanced-options { margin-bottom: $pad-large; - - .tooltip-wrap { - display: flex; - align-items: center; - width: 100%; - margin-bottom: $pad-large; - - .icon-tooltip { - span { - display: flex; - align-items: center; - } - p { - text-align: center; - } - } - - .form-field--input { - display: flex; - flex-direction: column; - width: 100%; - margin-right: $pad-small; - margin-bottom: 0; - } - - .form-field--checkbox { - display: flex; - flex-direction: column; - align-items: center; - margin-right: $pad-small; - margin-bottom: 0; - - .fleet-checkbox { - display: flex; - align-items: center; - } - } - } - - .tooltip-wrap--input { - .icon-tooltip { - span { - margin-top: 22px; - } - } - } } } diff --git a/frontend/pages/policies/ManagePoliciesPage/components/ManageAutomationsModal/ManageAutomationsModal.tsx b/frontend/pages/policies/ManagePoliciesPage/components/ManageAutomationsModal/ManageAutomationsModal.tsx index ff2d89aa34..b1b7ba6e3f 100644 --- a/frontend/pages/policies/ManagePoliciesPage/components/ManageAutomationsModal/ManageAutomationsModal.tsx +++ b/frontend/pages/policies/ManagePoliciesPage/components/ManageAutomationsModal/ManageAutomationsModal.tsx @@ -199,22 +199,20 @@ const ManageAutomationsModal = ({

Add a policy to turn on automations.

)} -
- -
+