[WEB-1578] chore: add max length validation for workspace slug in create workspace form. (#4778)
This commit is contained in:
parent
28fc8ccd0e
commit
cf13ac3116
2 changed files with 20 additions and 3 deletions
|
|
@ -179,7 +179,9 @@ export const CreateWorkspace: React.FC<Props> = (props) => {
|
||||||
onChange={(event) => {
|
onChange={(event) => {
|
||||||
onChange(event.target.value);
|
onChange(event.target.value);
|
||||||
setValue("name", event.target.value);
|
setValue("name", event.target.value);
|
||||||
setValue("slug", event.target.value.toLocaleLowerCase().trim().replace(/ /g, "-"));
|
setValue("slug", event.target.value.toLocaleLowerCase().trim().replace(/ /g, "-"), {
|
||||||
|
shouldValidate: true,
|
||||||
|
});
|
||||||
}}
|
}}
|
||||||
placeholder="Enter workspace name..."
|
placeholder="Enter workspace name..."
|
||||||
ref={ref}
|
ref={ref}
|
||||||
|
|
@ -202,6 +204,13 @@ export const CreateWorkspace: React.FC<Props> = (props) => {
|
||||||
<Controller
|
<Controller
|
||||||
control={control}
|
control={control}
|
||||||
name="slug"
|
name="slug"
|
||||||
|
rules={{
|
||||||
|
required: "Workspace slug is required",
|
||||||
|
maxLength: {
|
||||||
|
value: 48,
|
||||||
|
message: "Workspace slug should not exceed 48 characters",
|
||||||
|
},
|
||||||
|
}}
|
||||||
render={({ field: { value, ref, onChange } }) => (
|
render={({ field: { value, ref, onChange } }) => (
|
||||||
<div
|
<div
|
||||||
className={`relative flex items-center rounded-md border-[0.5px] px-3 ${
|
className={`relative flex items-center rounded-md border-[0.5px] px-3 ${
|
||||||
|
|
@ -230,6 +239,7 @@ export const CreateWorkspace: React.FC<Props> = (props) => {
|
||||||
{invalidSlug && (
|
{invalidSlug && (
|
||||||
<p className="text-sm text-red-500">{`URL can only contain ( - ), ( _ ) & alphanumeric characters.`}</p>
|
<p className="text-sm text-red-500">{`URL can only contain ( - ), ( _ ) & alphanumeric characters.`}</p>
|
||||||
)}
|
)}
|
||||||
|
{errors.slug && <span className="text-sm text-red-500">{errors.slug.message}</span>}
|
||||||
</div>
|
</div>
|
||||||
<hr className="w-full border-onboarding-border-100" />
|
<hr className="w-full border-onboarding-border-100" />
|
||||||
<div className="space-y-1">
|
<div className="space-y-1">
|
||||||
|
|
|
||||||
|
|
@ -148,7 +148,9 @@ export const CreateWorkspaceForm: FC<Props> = observer((props) => {
|
||||||
onChange={(e) => {
|
onChange={(e) => {
|
||||||
onChange(e.target.value);
|
onChange(e.target.value);
|
||||||
setValue("name", e.target.value);
|
setValue("name", e.target.value);
|
||||||
setValue("slug", e.target.value.toLocaleLowerCase().trim().replace(/ /g, "-"));
|
setValue("slug", e.target.value.toLocaleLowerCase().trim().replace(/ /g, "-"), {
|
||||||
|
shouldValidate: true,
|
||||||
|
});
|
||||||
}}
|
}}
|
||||||
ref={ref}
|
ref={ref}
|
||||||
hasError={Boolean(errors.name)}
|
hasError={Boolean(errors.name)}
|
||||||
|
|
@ -171,7 +173,11 @@ export const CreateWorkspaceForm: FC<Props> = observer((props) => {
|
||||||
control={control}
|
control={control}
|
||||||
name="slug"
|
name="slug"
|
||||||
rules={{
|
rules={{
|
||||||
required: "Workspace URL is required",
|
required: "Workspace slug is required",
|
||||||
|
maxLength: {
|
||||||
|
value: 48,
|
||||||
|
message: "Workspace slug should not exceed 48 characters",
|
||||||
|
},
|
||||||
}}
|
}}
|
||||||
render={({ field: { onChange, value, ref } }) => (
|
render={({ field: { onChange, value, ref } }) => (
|
||||||
<Input
|
<Input
|
||||||
|
|
@ -194,6 +200,7 @@ export const CreateWorkspaceForm: FC<Props> = observer((props) => {
|
||||||
{invalidSlug && (
|
{invalidSlug && (
|
||||||
<p className="text-sm text-red-500">{`URL can only contain ( - ), ( _ ) & alphanumeric characters.`}</p>
|
<p className="text-sm text-red-500">{`URL can only contain ( - ), ( _ ) & alphanumeric characters.`}</p>
|
||||||
)}
|
)}
|
||||||
|
{errors.slug && <span className="text-xs text-red-500">{errors.slug.message}</span>}
|
||||||
</div>
|
</div>
|
||||||
<div className="space-y-1 text-sm">
|
<div className="space-y-1 text-sm">
|
||||||
<span>
|
<span>
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue