Skip to content

Show clear signup error for usernames with underscores#2726

Draft
guaka wants to merge 8 commits into
mainfrom
cursor/signup-username-underscore-error-message
Draft

Show clear signup error for usernames with underscores#2726
guaka wants to merge 8 commits into
mainfrom
cursor/signup-username-underscore-error-message

Conversation

@guaka

@guaka guaka commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reject underscores earlier during signup
  • return a clear username-specific error message for underscore usernames
  • update signup validation coverage for underscore usernames

Validation

  • Server test task was attempted, but the local run did not reach tests because the native mmmagic module failed to load under the current Node runtime (ERR_DLOPEN_FAILED).

@guaka

guaka commented Apr 29, 2026

Copy link
Copy Markdown
Contributor Author

relates to #2723

@guaka

guaka commented May 2, 2026

Copy link
Copy Markdown
Contributor Author

let's try this simpler way first: d8fc934

@chmac

chmac commented May 2, 2026

Copy link
Copy Markdown
Member

Definitely seems like an easier option!

@guaka guaka deleted the branch main June 12, 2026 21:28
@guaka guaka closed this Jun 12, 2026
@guaka guaka reopened this Jun 12, 2026
@guaka guaka changed the base branch from master to main June 12, 2026 21:36
@guaka guaka added the run-e2e add this label to a PR if you want the e2e tests to run label Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e add this label to a PR if you want the e2e tests to run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants