Skip to content

[PROFILE-009] Align profile and onboarding validation - #69

Merged
chev0004 merged 4 commits into
developfrom
fix/profile-009
Jun 21, 2026
Merged

[PROFILE-009] Align profile and onboarding validation#69
chev0004 merged 4 commits into
developfrom
fix/profile-009

Conversation

@chev0004

Copy link
Copy Markdown
Owner

Outline

Summary

Profile and onboarding now validate their shared fields through common helpers so the profile form cannot save values the database would reject.


Changes & Enhancements

  • Type of Change:
    • Feature
    • Bugfix
    • Refactor
    • Chore
  • Changes:
    • (validation):
      • Add shared field helpers for language code, country, bio, and tags used by both the profile and onboarding schemas
    • (profile):
      • Validate primary and per-row target language codes, validate the country 2-letter shape, trim the bio before length checks, and trim tags before length and duplicate checks
    • (database):
      • Raise the profile tag limit constraint from 6 to 8 so it matches the premium tag cap the form already allows

Page / App Changes

Affected Next.js Routes (Local):

Before (Screenshot)
After (Screenshot)

Component / Storybook Changes

Affected Storybook Components (Local):

Before (Screenshot)
After (Screenshot)

Not Doing


Other (Remarks)

Includes drizzle migration 0006, which widens the profile tag check constraint; run db:migrate on deploy.


Checklists

  • Code is well-documented (comments, JSDoc, etc.).
  • Existing comments were updated as needed.
  • No out-of-scope changes are included.
  • Write TODO comments where future work is required.
  • Removed unnecessary debug code (e.g., console.log, debugger).
  • Self-reviewed and tested locally.

@chev0004 chev0004 self-assigned this Jun 21, 2026
@chev0004
chev0004 merged commit 11c6d38 into develop Jun 21, 2026
1 check passed
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.

1 participant