From e1643d1e3a2c17219943edd474c9d02eac6e492a Mon Sep 17 00:00:00 2001 From: Bryan Frimin Date: Wed, 11 Mar 2026 10:25:53 +0100 Subject: [PATCH] Add missing validation Signed-off-by: Bryan Frimin --- .../iam/organizations/people/_components/PersonForm.tsx | 2 +- pkg/iam/organization_service.go | 6 ++++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/apps/console/src/pages/iam/organizations/people/_components/PersonForm.tsx b/apps/console/src/pages/iam/organizations/people/_components/PersonForm.tsx index 234b93e60..3aadbe16e 100644 --- a/apps/console/src/pages/iam/organizations/people/_components/PersonForm.tsx +++ b/apps/console/src/pages/iam/organizations/people/_components/PersonForm.tsx @@ -1,4 +1,4 @@ -import { formatDatetime, getAssignableRoles, getRoles, peopleRoles } from "@probo/helpers"; +import { formatDatetime, getAssignableRoles, getRoles } from "@probo/helpers"; import { roles } from "@probo/helpers/src/roles"; import { useTranslate } from "@probo/i18n"; import { Button, Field, Input, Option } from "@probo/ui"; diff --git a/pkg/iam/organization_service.go b/pkg/iam/organization_service.go index e8db796eb..6159cb53e 100644 --- a/pkg/iam/organization_service.go +++ b/pkg/iam/organization_service.go @@ -213,7 +213,8 @@ func (cur *CreateUserRequest) Validate() error { v.CheckEach(cur.AdditionalEmailAddresses, "additional_email_addresses", func(index int, item any) { v.Check(item, fmt.Sprintf("additional_email_addresses[%d]", index), validator.Required(), validator.NotEmpty()) }) - v.Check(cur.Position, "position", validator.SafeText(TitleMaxLength)) + v.Check(cur.Kind, "kind", validator.SafeTextNoNewLine(NameMaxLength)) + v.Check(cur.Position, "position", validator.SafeTextNoNewLine(TitleMaxLength)) v.Check(cur.ContractStartDate, "contract_start_date", validator.Before(cur.ContractEndDate)) v.Check(cur.ContractEndDate, "contract_end_date", validator.After(cur.ContractStartDate)) @@ -228,7 +229,8 @@ func (upr *UpdateUserRequest) Validate() error { v.CheckEach(upr.AdditionalEmailAddresses, "additional_email_addresses", func(index int, item any) { v.Check(item, fmt.Sprintf("additional_email_addresses[%d]", index), validator.Required(), validator.NotEmpty()) }) - v.Check(upr.Position, "position", validator.SafeText(TitleMaxLength)) + v.Check(upr.Kind, "kind", validator.SafeTextNoNewLine(NameMaxLength)) + v.Check(upr.Position, "position", validator.SafeTextNoNewLine(TitleMaxLength)) v.Check(upr.ContractStartDate, "contract_start_date", validator.Before(upr.ContractEndDate)) v.Check(upr.ContractEndDate, "contract_end_date", validator.After(upr.ContractStartDate))