Skip to content

Implement new onboarding modal #346#353

Merged
leeyandrea merged 7 commits intomainfrom
Implement-New-Onboarding-Modal-#346
Feb 1, 2026
Merged

Implement new onboarding modal #346#353
leeyandrea merged 7 commits intomainfrom
Implement-New-Onboarding-Modal-#346

Conversation

@leeyandrea
Copy link
Contributor

@leeyandrea leeyandrea commented Jan 30, 2026

  • Made many styling changes on the Onboarding form

  • added 'X' function to clear inputs

  • added form control on Majors (made required)

  • turned the "have you co-op ed before" yes/no buttons into a checkbox to match designs

Onboarding From has new designs

  • changed the header from Create a cooper account to “lets get setup”,
  • deleted “required” subheader to match design
  • switched the order of month and year graduation date boxes to match design
  • replaced the “have you co-op ed before” yes/no buttons with a checkbox to match designs
  • changed styling of Finish button
  • removed className="max-w-72” from FormItem so all the inputs fill to the form
  • deleted required from last question checkbox
  • added form control and msg for major
  • added an onClear and X button to input component for clearing inputs
  • dialog.tsx (onboarding) changed padding to match design
  • changed input styles to match design
  • changed dialog component style to match design
  • changed select component style to match design

Closes #346

STILL LEFT TO DO:

  • make the dropdowns use the search as the field instead of 2 clicks (same fucntion as the benefits dropdown int he review form)
  • make the graduation year a dropdown too to match the graduation month
  • clicking the text on the checkbox at the bottom also checks it
  • adding minors dropdown logic (like majors)

How has this been tested?

  • changing stylings of major and graduation month dropdown component affect the styling of the review form dropdowns, using the same combobox component

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Database migration
    • Ran pnpm db:generate and verified generated SQL migration files in packages/db/drizzle

Checklist:

  • My code follows the code style of this project.
  • I have moved the ticket to "In Review"
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@vercel
Copy link

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cooper Ready Ready Preview, Comment Feb 1, 2026 3:46pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
cooper-auth Skipped Skipped Feb 1, 2026 3:46pm
cooper-docs Skipped Skipped Feb 1, 2026 3:46pm

Request Review

@vercel vercel bot temporarily deployed to Preview – cooper-auth January 30, 2026 00:24 Inactive
@vercel vercel bot temporarily deployed to Preview – cooper-docs January 30, 2026 00:24 Inactive
@leeyandrea leeyandrea marked this pull request as ready for review January 30, 2026 00:26
Copy link
Contributor

@Tiffybin Tiffybin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome sauce. Was wondering if you noticed that the placeholder for email domains was @example.com. Should it be @husky.neu.edu since only northeastern accounts can sign up? Kinda nitpicky doe.

Copy link
Collaborator

@gpalmer27 gpalmer27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just two very small changes then it should be good to merge

@vercel vercel bot temporarily deployed to Preview – cooper-auth February 1, 2026 15:45 Inactive
@vercel vercel bot temporarily deployed to Preview – cooper-docs February 1, 2026 15:45 Inactive
@leeyandrea leeyandrea merged commit c3f06c7 into main Feb 1, 2026
10 checks passed
@leeyandrea leeyandrea deleted the Implement-New-Onboarding-Modal-#346 branch February 1, 2026 15:50
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.

Implement New Onboarding Modal

3 participants