From 597fde977fb027fe89b79e529b3b587d4cd03eda Mon Sep 17 00:00:00 2001 From: Darin Kelkhoff Date: Fri, 20 Sep 2024 10:44:26 -0500 Subject: [PATCH] CE-1727 - in Yup required message, show 'This field' instead of 'undefined' if field doesn't have a label --- src/qqq/components/forms/DynamicFormUtils.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qqq/components/forms/DynamicFormUtils.ts b/src/qqq/components/forms/DynamicFormUtils.ts index 310bb86..89331a9 100644 --- a/src/qqq/components/forms/DynamicFormUtils.ts +++ b/src/qqq/components/forms/DynamicFormUtils.ts @@ -136,11 +136,11 @@ class DynamicFormUtils // the "nullable(true)" here doesn't mean that you're allowed to set the field to null... // // rather, it's more like "null is how empty will be treated" or some-such... // //////////////////////////////////////////////////////////////////////////////////////////// - return (Yup.string().required(`${field.label} is required.`).nullable(true)); + return (Yup.string().required(`${field.label ?? "This field"} is required.`).nullable(true)); } else { - return (Yup.string().required(`${field.label} is required.`)); + return (Yup.string().required(`${field.label ?? "This field"} is required.`)); } } return (null);