fix: error display

jaaydenh · jaaydenh · commit d3b64b2f70d2 · 2025-01-27T17:03:14.000Z
diff --git a/site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpGroupSyncForm.tsx b/site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpGroupSyncForm.tsx
@@ -164,9 +164,9 @@ export const IdpGroupSyncForm = ({
 							</p>
 						</div>
 					</div>
-					{form.errors && (
+					{form.errors.field || form.errors.regex_filter && (
 						<p className="text-content-danger text-sm m-0">
-							{form?.errors?.field || form?.errors?.regex_filter}
+							{form.errors.field || form.errors.regex_filter}
 						</p>
 					)}
 				</div>
@@ -253,9 +253,9 @@ export const IdpGroupSyncForm = ({
 						</Button>
 					</div>
 				</div>
-				{form.errors && (
+				{form.errors.mapping && (
 					<p className="text-content-danger text-sm m-0">
-						{Object.values(form?.errors?.mapping || {})}
+						{Object.values(form.errors.mapping || {})}
 					</p>
 				)}
 				<div className="flex flex-col">
diff --git a/site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpRoleSyncForm.tsx b/site/src/pages/ManagementSettingsPage/IdpSyncPage/IdpRoleSyncForm.tsx
@@ -127,9 +127,9 @@ export const IdpRoleSyncForm = ({
 						If empty, role sync is deactivated
 					</p>
 				</div>
-				{form.errors && (
+				{form.errors.field && (
 					<p className="text-content-danger text-sm m-0">
-						{form?.errors?.field}
+						{form.errors.field}
 					</p>
 				)}
 				<div className="flex flex-row gap-2 justify-between items-start">
@@ -197,9 +197,9 @@ export const IdpRoleSyncForm = ({
 						</Button>
 					</div>
 				</div>
-				{form.errors && (
+				{form.errors.mapping && (
 					<p className="text-content-danger text-sm m-0">
-						{Object.values(form?.errors?.mapping || {})}
+						{Object.values(form.errors.mapping || {})}
 					</p>
 				)}
 				<IdpMappingTable type="Role" rowCount={roleMappingCount}>
