Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: fixed pushing data to supabase. changed inputdropdown so ch… #39

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

me-liu
Copy link
Contributor

@me-liu me-liu commented Dec 8, 2024

…oices persist. minimal styling changes

🍞 What's new in this PR

🥐 Description

  • fixed supabase pushing so things show up on table now
  • changed inputdropdown so that you can go back and see your selections
  • moved the button down for the review screen (but it looks pretty much the same)

🥖 Screenshots

N/A

🥪 How to review

  • all at once!

🥧 Next steps

  • If you don't confirm your email before doing signup, it will not push to the proper user ID when submitting to supabase. there needs to be a check to ensure the email that you sign up with is confirmed before moving on to onboarding

🥞 Relevant links

🥨 Online sources

🥯 Related PRs

CC: @celinechoiii

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.

1 participant