feat: Mentee registration - Final#247
Merged
womencodingcommunity merged 10 commits intomainfrom Mar 7, 2026
Merged
Conversation
Part of #230 Add components for Steps 1-3 of the mentee registration form. Included form inputs, validations, and supporting tests to ensure functionality aligns with design.
Simplify form schema and UI by eliminating the unused `cycleYear` field. Updated validations and adjusted mentorship type section to reflect current form logic.
…tants, update form schema and experience options Removed redundant `Software Development` group in `TECHNICAL_AREA_GROUPS`, adjusted `Slack display name` and `LinkedIn URL` validation, and refined experience dropdown labels for better clarity.
Replace the placeholder console.log and fake promise delay with a real /api/mentee-registration Next.js route that proxies the payload to the backend platform endpoint. The page now renders a success screen on completion and surfaces API error messages as an inline alert. Schema validation was tightened (50-char minimum for whyMentor, availableHsMonth capped at 224, isWomen field added). Structured logging via bs-logger replaces console.* calls in api.ts, and merge conflicts in pnpm-lock.yaml are resolved.
The `global` object is Node.js-specific, while `globalThis` is the standard ECMAScript cross-platform equivalent. Using `globalThis` improves compatibility and follows modern best practices. Also removes a stale comment and reformats an assertion for better readability.
dricazenck
requested review from
Amenawon,
IdayatSanni,
aaguirre-rdit and
nora-weisser
Add project conventions documentation (CLAUDE.md) covering the pre-commit hook, Prettier config, ESLint rules, and testing standards. Also apply Prettier auto-formatting to several components and tests that had lines exceeding the 80-char print width, keeping the codebase consistent with the enforced style config.
Fix all ESLint violations flagged after enabling stricter linting: reorder imports to satisfy the import/order rule (external before internal, with newlines between groups), escape an unescaped apostrophe in Step5Review, replace console.log/error calls with the bs-logger utility to satisfy the no-console rule, and normalise import paths to use the configured TypeScript aliases.
…tration_complete feat: mentee registration API submission
The @schemas path alias is not configured in tsconfig, causing a runtime resolution error. Revert to the working relative-style path `schemas/mentorSchema`. Also revert the bs-logger usage in onSubmit since it was incorrectly introduced — the logger import was removed but the call remained, which would cause a build error.
Reorganise test files from co-located __tests__ folders (next to source) into a centralised src/__tests__/ directory, mirroring the api/ and pages/ sub-structure. This aligns with the project's preferred test layout and makes test files easier to discover across the codebase.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add components for Steps 1-3 of the mentee registration form. Included form inputs, validations, and supporting tests to ensure functionality aligns with design.
Type
Screenshots
After:
Mentee Page 1:
Mentor Selection:
Before:
Pull request checklist
Please check if your PR fulfills the following requirements: