Skip to content

fix: allow special characters in project name#9341

Open
Suryacodeshere wants to merge 1 commit into
makeplane:previewfrom
Suryacodeshere:fix-project-name-special-chars
Open

fix: allow special characters in project name#9341
Suryacodeshere wants to merge 1 commit into
makeplane:previewfrom
Suryacodeshere:fix-project-name-special-chars

Conversation

@Suryacodeshere

@Suryacodeshere Suryacodeshere commented Jul 1, 2026

Copy link
Copy Markdown

Resolves #9226 and #9280.

Description

The project name (title) is a user-facing label and should be allowed to contain special characters such as hyphens -, ampersands &, etc. However, the serializers in the backend API and App endpoints were validating the project
ame against Project.FORBIDDEN_IDENTIFIER_CHARS_PATTERN (which is meant only for project identifiers to ensure they are valid prefix patterns). This caused project creation and update requests containing special characters in their name to fail with PROJECT_NAME_CANNOT_CONTAIN_SPECIAL_CHARACTERS.

This change removes the validation of project names against the forbidden identifier characters regex, while keeping the validation in place for project identifiers.

Changes

  • Removed name validation using Project.FORBIDDEN_IDENTIFIER_CHARS_PATTERN from ProjectCreateSerializer, ProjectUpdateSerializer, and ProjectSerializer in �pps/api/plane/api/serializers/project.py.
  • Removed name validation using Project.FORBIDDEN_IDENTIFIER_CHARS_PATTERN from ProjectSerializer.validate_name in �pps/api/plane/app/serializers/project.py.

Summary by CodeRabbit

  • Bug Fixes
    • Project names no longer reject certain special characters during creation or editing.
    • Identifier validation remains unchanged, so project identifiers still follow the existing character rules.

@CLAassistant

CLAassistant commented Jul 1, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b7d329be-18b4-4bfd-b743-0a25e7f6e63f

📥 Commits

Reviewing files that changed from the base of the PR and between 7fbf14a and b53b505.

📒 Files selected for processing (2)
  • apps/api/plane/api/serializers/project.py
  • apps/api/plane/app/serializers/project.py
💤 Files with no reviewable changes (2)
  • apps/api/plane/api/serializers/project.py
  • apps/api/plane/app/serializers/project.py

📝 Walkthrough

Walkthrough

Removes forbidden-character validation for project name fields in three API serializer methods and one app serializer's validate_name method, while identifier validation against FORBIDDEN_IDENTIFIER_CHARS_PATTERN and name uniqueness checks remain unchanged.

Changes

Project name validation removal

Layer / File(s) Summary
API serializer name validation removal
apps/api/plane/api/serializers/project.py
ProjectCreateSerializer.validate, ProjectUpdateSerializer.update, and ProjectSerializer.validate no longer check name against the forbidden-character pattern, retaining only identifier validation.
App serializer name validation removal
apps/api/plane/app/serializers/project.py
ProjectSerializer.validate_name no longer raises PROJECT_NAME_CANNOT_CONTAIN_SPECIAL_CHARACTERS, proceeding directly to workspace name uniqueness checks.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • makeplane/plane#9059: Adds frontend handling for the same PROJECT_NAME_CANNOT_CONTAIN_SPECIAL_CHARACTERS error code being removed on the backend in this PR.

Suggested reviewers: sriramveeraghanta, Rahulcheryala

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly states the core change: allowing special characters in project names.
Description check ✅ Passed The description is mostly complete and explains the bug, fix, and affected files.
Linked Issues check ✅ Passed The changes remove the invalid name regex check, matching the linked bug's expected project creation behavior.
Out of Scope Changes check ✅ Passed The modified serializers stay focused on project-name validation and add no unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Suryacodeshere

Copy link
Copy Markdown
Author

@dheeru0198 @pablohashescobar Whenever you have time. Could you please review it? Thank You!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug]: Project creation fails with special characters in project name

2 participants