Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the hacker registration flow UI/UX by introducing a new multi-step “details” experience (role + skill level) with new mascot artwork, and removes the old detail-selection implementation and its legacy assets.
Changes:
- Add new hacker registration “details” flow components (
RoleStage,LevelStage, newDetailForm) and new register-page copy. - Replace legacy detail form + SCSS module with the new staged UI (background now handled within stages).
- Add new public register mascots and remove unused legacy login mascots.
Reviewed changes
Copilot reviewed 8 out of 19 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| public/hackers/register/product-frog.svg | Adds new “Product” role mascot asset for the register details UI. |
| public/hackers/register/explorer-duck.svg | Adds new “Other” role mascot asset for the register details UI. |
| public/hackers/register/experienced-frog.svg | Adds new “Experienced” level mascot asset for the register details UI. |
| public/hackers/register/designer-bunny.svg | Adds new “Designer” role mascot asset for the register details UI. |
| public/hackers/register/beginner-frog.svg | Adds new “Beginner” level mascot asset for the register details UI. |
| public/hackers/login/pm_froggy.svg | Removes legacy login mascot asset (no longer used by the new flow). |
| public/hackers/login/other_ducky.svg | Removes legacy login mascot asset (no longer used by the new flow). |
| public/hackers/login/developer_cow.svg | Removes legacy login mascot asset (no longer used by the new flow). |
| public/hackers/login/designer_bunny.svg | Removes legacy login mascot asset (no longer used by the new flow). |
| app/(pages)/(hackers)/register/page.tsx | Updates imports and refreshes title/subtitle copy for the register page. |
| app/(pages)/(hackers)/register/details/page.tsx | Switches details page to render the new staged DetailForm directly. |
| app/(pages)/(hackers)/_components/AuthForms/register/RoleStage.tsx | Adds new role selection stage UI (cards + navigation). |
| app/(pages)/(hackers)/_components/AuthForms/register/RegisterForm.tsx | Adds new register form implementation for hackers using shared AuthForm. |
| app/(pages)/(hackers)/_components/AuthForms/register/LevelStage.tsx | Adds new skill level selection stage UI (cards + navigation + error state). |
| app/(pages)/(hackers)/_components/AuthForms/register/DetailForm.tsx | Adds new staged details form that persists position + is_beginner. |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.tsx | Removes legacy details form implementation (radio grid + beginner checkbox). |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.module.scss | Removes legacy SCSS module styling for the deleted details form. |
| app/(pages)/(hackers)/_components/AuthFormBackground/AuthFormBackground.tsx | Adds showAngelCow prop to optionally hide the angel cow mascot. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
app/(pages)/(hackers)/_components/AuthForms/register/LevelStage.tsx
Outdated
Show resolved
Hide resolved
app/(pages)/(hackers)/_components/AuthForms/register/RoleStage.tsx
Outdated
Show resolved
Hide resolved
app/(pages)/(hackers)/_components/AuthForms/register/RoleStage.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the hacker registration flow and visuals, introducing a new multi-step “details” experience (role + skill level) and migrating mascot assets from the old login set to new register-specific SVGs.
Changes:
- Added new register mascot SVG assets and removed legacy login-role SVG assets.
- Reworked hacker “details” registration into a two-stage client flow (RoleStage → LevelStage) that persists to the user record.
- Updated AuthFormBackground to support conditionally hiding the “Angel Cow” mascot.
Reviewed changes
Copilot reviewed 8 out of 19 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| public/hackers/register/product-frog.svg | New “Product” role mascot asset for register flow |
| public/hackers/register/explorer-duck.svg | New “Other” role mascot asset for register flow |
| public/hackers/register/experienced-frog.svg | New “Experienced” level mascot asset |
| public/hackers/register/designer-bunny.svg | New “Designer” role mascot asset for register flow |
| public/hackers/register/beginner-frog.svg | New “Beginner” level mascot asset |
| public/hackers/login/pm_froggy.svg | Removed legacy login-role mascot asset |
| public/hackers/login/other_ducky.svg | Removed legacy login-role mascot asset |
| public/hackers/login/developer_cow.svg | Removed legacy login-role mascot asset |
| public/hackers/login/designer_bunny.svg | Removed legacy login-role mascot asset |
| app/(pages)/(hackers)/register/page.tsx | Updated register page copy + updated imports to new RegisterForm path |
| app/(pages)/(hackers)/register/details/page.tsx | Simplified page to render the new staged DetailForm directly |
| app/(pages)/(hackers)/_components/AuthForms/register/RoleStage.tsx | New role selection stage UI |
| app/(pages)/(hackers)/_components/AuthForms/register/LevelStage.tsx | New skill level selection stage UI |
| app/(pages)/(hackers)/_components/AuthForms/register/DetailForm.tsx | New staged details controller that updates user position + is_beginner |
| app/(pages)/(hackers)/_components/AuthForms/register/RegisterForm.tsx | New hacker register form implementation under AuthForms/register |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.tsx | Removed legacy details form implementation |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.module.scss | Removed legacy styling tied to removed details form |
| app/(pages)/(hackers)/_components/AuthFormBackground/AuthFormBackground.tsx | Added showAngelCow?: boolean prop and conditional rendering |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the hacker sign-up flow to avoid “400” failures during registration (by ensuring server-action return values are serializable) and refreshes the hacker registration UX with a new two-step “role + skill level” flow and new mascot assets.
Changes:
- Make
RegisterActionreturn a serializable success payload (avoiding returning Mongo documents from server actions). - Replace the old hacker details form with a new staged
ChooseRole→ChooseLevelflow using new/public/hackers/register/*SVG assets. - Update hacker register page copy/imports and add an optional
showAngelCowtoggle toAuthFormBackground.
Reviewed changes
Copilot reviewed 9 out of 20 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| public/hackers/register/product-frog.svg | Adds new “Product” mascot SVG used in role selection. |
| public/hackers/register/explorer-duck.svg | Adds new “Other” mascot SVG used in role selection. |
| public/hackers/register/experienced-frog.svg | Adds new “Experienced” mascot SVG used in level selection. |
| public/hackers/register/designer-bunny.svg | Adds new “Designer” mascot SVG used in role selection. |
| public/hackers/register/beginner-frog.svg | Adds new “Beginner” mascot SVG used in level selection. |
| public/hackers/login/pm_froggy.svg | Removes unused legacy login mascot SVG. |
| public/hackers/login/other_ducky.svg | Removes unused legacy login mascot SVG. |
| public/hackers/login/developer_cow.svg | Removes unused legacy login mascot SVG. |
| public/hackers/login/designer_bunny.svg | Removes unused legacy login mascot SVG. |
| app/(pages)/(hackers)/register/page.tsx | Updates imports and register page title/subtitle copy; uses new RegisterForm location. |
| app/(pages)/(hackers)/register/details/page.tsx | Simplifies details page wrapper; renders new staged DetailForm directly. |
| app/(pages)/(hackers)/_components/AuthForms/register/RegisterForm.tsx | Adds new hacker register form wrapper; integrates with AuthForm and redirects to details step. |
| app/(pages)/(hackers)/_components/AuthForms/register/DetailForm.tsx | Adds staged client flow handling role/level selection and saving to user profile. |
| app/(pages)/(hackers)/_components/AuthForms/register/ChooseRole.tsx | Adds role selection UI using new mascots and AuthFormBackground. |
| app/(pages)/(hackers)/_components/AuthForms/register/ChooseLevel.tsx | Adds level selection UI using new mascots and AuthFormBackground. |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.tsx | Removes legacy single-page details form implementation. |
| app/(pages)/(hackers)/_components/AuthForms/DetailForm.module.scss | Removes legacy SCSS tied to the deleted details form. |
| app/(pages)/(hackers)/_components/AuthFormBackground/AuthFormBackground.tsx | Adds showAngelCow prop to optionally hide the “Angel Cow” header element. |
| app/(api)/_actions/auth/register.ts | Wraps @datalib/auth/register to return a serializable success result and catch thrown errors. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
michelleyeoh
left a comment
There was a problem hiding this comment.
Pretty nice, next time let me know if there's issues finding the correct page ;-;
Changes:
- Moved role and level stage & integrated into existing detail/register form
- Deleted old register file & assets
- Wrapped register in AuthFormBackground
- Also fixed server error with register.ts
No description provided.