Skip to content

Rebeccals onboard#43

Merged
larrylaa merged 6 commits intodevfrom
Rebeccals-Onboard
Apr 12, 2025
Merged

Rebeccals onboard#43
larrylaa merged 6 commits intodevfrom
Rebeccals-Onboard

Conversation

@Rebeccals1
Copy link
Contributor

Changes made to onboard page:

✅ Validation

  • Added a validation system where each step can expose its own validate() function via forwardRef.
  • OnboardPage calls validate() on the current step before allowing a transition to the next step.

Required that:

  • CreateProfile has a name and email filled out.
  • Location has at least one location selected.
  • Roles requires at least one role.
  • Interest checks if any interest is selected.
  • Skills needs at least one skill.

🧭 Progress Handling

  • Replaced the default step indicators (dots) with a progress bar.
  • Progress bar updates visually as the user moves through steps.

🌀 Animated Transitions

  • Added animated transitions between onboarding steps (e.g., fade or slide) for a smoother UX.

📋 Backend Integration

  • Collected all form data across steps into formData in OnboardPage.
  • On Submit, made a POST request to a Django backend endpoint with the full form data (Needs testing).

@Rebeccals1 Rebeccals1 requested a review from larrylaa April 8, 2025 21:33
Copy link
Contributor

@larrylaa larrylaa left a comment

Choose a reason for hiding this comment

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

The frontend for the onboarding flow looks really nice, however I am encountering an issue where my screen view goes to the mobile view despite being on desktop.

Can you fix this?

The backend API call doesn't work right now because I don't have it implemented, so I will be handling that. Just fix the screen shrinking.

I've attached a photo below.

Thanks, and nice job!
Shrink

@Rebeccals1
Copy link
Contributor Author

Ok, I updated the branch. The issue should be fixed.

@larrylaa larrylaa merged commit b02100a into dev Apr 12, 2025
4 of 5 checks passed
larrylaa added a commit that referenced this pull request Apr 20, 2025
* fix: like/follow routes

* feat: Rebeccals onboard (#43)

* Updated Onboarding Page

* More onboarding updates

* Fixed onboard container being too small. Added link to home slideshow.

* changed onboard so its public

* chore: workflow double run fix

* chore: double unit test run fix

---------

Co-authored-by: Rebecca Smith <2145912+Rebeccals@users.noreply.github.com>
Co-authored-by: Larry La <larryquocla@gmail.com>

* chore: drop db to fix everything

* feat: profile page (#44)

* feat. like and follow buttons

* feat. new profile page draft

---------

Co-authored-by: luisd101 <luisdominguezrock@gmail.com>

* feat: onboarding flow + user auth basics (#45)

* feat: edit user schema to support onboarding

* feat: user onboard backend route

* feat: check if onboarded

* chore: eslint rule update

* feat: working onboarding flow

* feat: added the match score algorithm

* feat: student profile backend (#46)

* feat: react routing user id

* chore: api call

* feat: projects showing

* feaT: working scroll

* feat: project permissions

* feat: completed profile page

* chore: email

* chore: update the edit modal

* chore: passed in userdata

* feat: update works

* fix: major not showing

* chore: final touch up

* fix: placeholder profile image, redirect to correct page on back button

* chore: pull data and pass into matchscoreutils

* chore: add for home page also

* chore: abstract user error msg

* fix: MatchScoreUtils.js to calculate new user values as well as correctly outputs weighted scores

* feat: new calculation logic

* chore: make tags be added on project creation

* chore: make score sseem more natural

* fix: page error on empty tags

* fix: sizing broke

* feat: about page (#49)

* Updated About page

* chore: hide form

---------

Co-authored-by: Rebecca Smith <2145912+Rebeccals@users.noreply.github.com>
Co-authored-by: Larry La <larryquocla@gmail.com>

---------

Co-authored-by: Rebecca Smith <2145912+Rebeccals1@users.noreply.github.com>
Co-authored-by: Rebecca Smith <2145912+Rebeccals@users.noreply.github.com>
Co-authored-by: luisd101 <luisdominguezrock@gmail.com>
Co-authored-by: ColumnSkunky <wfgs03@gmail.com>
Co-authored-by: William Garcia <102125255+ColumnSkunky@users.noreply.github.com>
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.

2 participants