Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: project identifier validation #1643

Merged
merged 5 commits into from
Jul 25, 2023
Merged

fix: project identifier validation #1643

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
33294bb
fix: project identifier validation
anmolsinghbhatia Jul 24, 2023
d59de7e
feat: alphanumric identifier validation
anmolsinghbhatia Jul 24, 2023
03c5f3c
chore: code refactor
anmolsinghbhatia Jul 24, 2023
1899308
Merge branch 'develop' of https://github.com/makeplane/plane into fix…
aaryan610 Jul 25, 2023
38da97e
refactor: project identifier change function
aaryan610 Jul 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
23 changes: 20 additions & 3 deletions apps/app/components/project/create-project-modal.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,13 @@ export const CreateProjectModal: React.FC<Props> = ({ isOpen, setIsOpen, user })

useEffect(() => {
if (projectName && isChangeIdentifierRequired)
setValue("identifier", projectName.replace(/ /g, "").toUpperCase().substring(0, 3));
setValue(
"identifier",
projectName
.replace(/[^a-zA-Z0-9]/g, "")
.toUpperCase()
.substring(0, 3)
);
}, [projectName, projectIdentifier, setValue, isChangeIdentifierRequired]);

useEffect(() => () => setIsChangeIdentifierRequired(true), [isOpen]);
Expand Down Expand Up @@ -148,6 +154,16 @@ export const CreateProjectModal: React.FC<Props> = ({ isOpen, setIsOpen, user })
});
};

const handleIdentifierChange = (event: React.ChangeEvent<HTMLInputElement>) => {
const { value } = event.target;

const alphanumericValue = value.replace(/[^a-zA-Z0-9]/g, "");
const FormattedValue = alphanumericValue.toUpperCase();
aaryan610 marked this conversation as resolved.
Show resolved Hide resolved

setValue("identifier", FormattedValue);
setIsChangeIdentifierRequired(false);
};

const options = workspaceMembers?.map((member) => ({
value: member.member.id,
query:
Expand Down Expand Up @@ -288,11 +304,12 @@ export const CreateProjectModal: React.FC<Props> = ({ isOpen, setIsOpen, user })
placeholder="Identifier"
error={errors.identifier}
register={register}
onChange={() => setIsChangeIdentifierRequired(false)}
onChange={handleIdentifierChange}
validations={{
required: "Identifier is required",
validate: (value) =>
/^[A-Z]+$/.test(value) || "Identifier must be in uppercase.",
/^[A-Z0-9]+$/.test(value.toUpperCase()) ||
"Identifier must be in uppercase.",
minLength: {
value: 1,
message: "Identifier must at least be of 1 character",
Expand Down
12 changes: 11 additions & 1 deletion apps/app/pages/[workspaceSlug]/projects/[projectId]/settings/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,15 @@ const GeneralSettings: NextPage = () => {
else await updateProject(payload);
};

const handleIdentifierChange = (event: React.ChangeEvent<HTMLInputElement>) => {
const { value } = event.target;

const alphanumericValue = value.replace(/[^a-zA-Z0-9]/g, "");
const FormattedValue = alphanumericValue.toUpperCase();

setValue("identifier", FormattedValue);
};

const currentNetwork = NETWORK_CHOICES.find((n) => n.key === projectDetails?.network);

return (
Expand Down Expand Up @@ -303,10 +312,11 @@ const GeneralSettings: NextPage = () => {
error={errors.identifier}
register={register}
placeholder="Enter identifier"
onChange={handleIdentifierChange}
validations={{
required: "Identifier is required",
validate: (value) =>
/^[A-Z]+$/.test(value) || "Identifier must be uppercase text.",
/^[A-Z0-9]+$/.test(value.toUpperCase()) || "Identifier must be in uppercase.",
minLength: {
value: 1,
message: "Identifier must at least be of 1 character",
Expand Down