Skip to content

Conversation

@aasthas23
Copy link
Contributor

Created the pages for the survey after the used signs up!

@ryanpolasky
Copy link
Contributor

image Can we center these two buttons??

@ryanpolasky
Copy link
Contributor

ryanpolasky commented Dec 19, 2025

image

Can we catch this on the same page? i.e. disable the button or block the request & add a modal that says "Must select X" based on what is missing? It also looks like not every selection is required across the pages, but all of them should be required. They should not be able to progress unless they've made all the selections.

@ryanpolasky
Copy link
Contributor

Looks like the route isn't protected -> A user can simply go to meteormate.com (or wherever we host it, localhost:3000 in dev) and then add /onboarding/createProfile to start the flow, which bypasses all user auth. Totally fine for dev (actually kind of helpful since it lets me test it without an account), but we 1000% need this route protected.

Copy link
Contributor

@ryanpolasky ryanpolasky left a comment

Choose a reason for hiding this comment

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

See the comments on the issue for review :) Code looks mostly good, just some logical things that need changing before a full code review.

@ryanpolasky ryanpolasky self-assigned this Dec 23, 2025
@ryanpolasky ryanpolasky added the Type: Feature Request New feature or request label Dec 23, 2025
@ryanpolasky ryanpolasky added this to the MVP milestone Dec 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature Request New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

task: Connect sign-up page with the survey page task: Convert .pngs to .webps feat: Create profile page

4 participants