Skip to content

fix: show specific error messages for waitlist registration (#673)#897

Open
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/registration-error-messages-673
Open

fix: show specific error messages for waitlist registration (#673)#897
NewCoder3294 wants to merge 1 commit intokoala73:mainfrom
NewCoder3294:fix/registration-error-messages-673

Conversation

@NewCoder3294
Copy link
Contributor

Summary

  • Improved error handling for the "Reserve Your Spot" waitlist registration
  • Users now see specific messages instead of generic "Registration failed" for:
    • Rate limiting (429) -> "Too many attempts. Please try again in an hour."
    • Service unavailable (503) -> "Registration service is temporarily unavailable."
    • Network errors (fetch failure) -> "Network error. Check your connection and try again."
  • Added i18n keys (errorRateLimit, errorServiceUnavailable, errorNetwork) in en.json
  • Other locales fall back to English via i18next fallbackLng: 'en'

Test plan

  • Register with new email -> success message
  • Register with same email again -> "already registered" message
  • Simulate rate limiting (6+ requests in 1 hour) -> "Too many attempts" message
  • Simulate service unavailable (503) -> "Registration service is temporarily unavailable"
  • Disconnect network, then register -> "Network error" message
  • TypeScript type check passes (npx tsc --noEmit)

Closes #673

🤖 Generated with Claude Code

)

Instead of a generic 'Registration failed' for all errors, now shows:
- Already registered status (already handled, unchanged)
- Rate limit exceeded (429) message
- Service unavailable (503) message
- Network error message (catch block)

Added i18n keys: errorRateLimit, errorServiceUnavailable, errorNetwork

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Mar 3, 2026

@NewCoder3294 is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@koala73 koala73 added Low Value Trivial, unnecessary, or not aligned with project needs Ready to Merge PR is mergeable, passes checks, and adds value labels Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Low Value Trivial, unnecessary, or not aligned with project needs Ready to Merge PR is mergeable, passes checks, and adds value

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to register for "Reserve Your Spot" waitlist.

2 participants