Skip to content

Conversation

@paanSinghCoder
Copy link
Contributor

@paanSinghCoder paanSinghCoder commented Nov 18, 2025

Summary

This PR:

  • Changes the validation of all 4 delete confirm dialogs from name to title.
  • Changes the error messages
  • Minor prop issue.

Changes

Organization Deletion (general/delete.tsx)

  • Changed validation from organization.name (URL/slug) to organization.title (display name)
  • Updated input label and placeholder to request organization name instead of URL
  • Updated error message to "The organization name does not match"

Teams Deletion (teams/delete.tsx)

  • Changed validation from team.name to team.title
  • Updated input label to "Please enter the title of the team to confirm."
  • Updated placeholder to "Enter the team title"
  • Updated error message to "Team title does not match"

Projects Deletion (project/delete.tsx)

  • Changed validation from project.name to project.title
  • Updated input label to "Please enter the title of the project to confirm."
  • Updated placeholder to "Enter the project title"
  • Updated error message to "Project title does not match"

Domain Deletion (domain/delete.tsx)

  • Updated input label to "Please enter the domain name to confirm."
  • Updated placeholder to "Enter the domain name"
  • Updated error message to "Domain name does not match"
  • Fixed acknowledgment text to reference "domain data" instead of "team data"

Technical Details

Test Plan

  • Manual testing completed
  • Build and type checking passes

@vercel
Copy link

vercel bot commented Nov 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
frontier Ready Ready Preview Comment Nov 21, 2025 4:00am

@coveralls
Copy link

coveralls commented Nov 18, 2025

Pull Request Test Coverage Report for Build 19559459153

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 37.603%

Totals Coverage Status
Change from base Build 19528138208: 0.0%
Covered Lines: 15636
Relevant Lines: 41582

💛 - Coveralls

async function onSubmit(data: any) {
if (!organization?.id) return;
if (data.name !== organization.name)
if (data.name !== organization.title)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets change the form field naming as well to title ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@rohilsurana
Copy link
Member

PR title should be opposite, name to title ?

if (!organization?.id || !projectId) return;
if (data.name !== project?.name)
return setError('name', { message: 'project name is not same' });
if (data.name !== project?.title)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done


if (data.name !== team?.name)
return setError('name', { message: 'team name is not same' });
if (data.name !== team?.title)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@paanSinghCoder paanSinghCoder changed the title fix: change delete confirm input field from title to name fix: change delete confirm input field from name to title Nov 20, 2025
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.

4 participants