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

implement components and basic onboarding pages #137

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

Conversation

josephshenq
Copy link
Collaborator

What does this Pull Request change?

Creates some common components such as CheckboxCard, Modal, etc. Also build 2nd page (Household) and 4th page (Request Items) of onboarding pages.

However, styling all the input fields to put them in the middle of the (total screen - sidebar) is hard and behave different on mobile/web. This is not fully done yet.

Besides these, the error message for 2nd page (Household) is not done yet. The error message won't automatically show when users enter inputs for some reason.

Important Changes

  • Installed Jotai for atoms usage (I used atoms for recording the global state for form values, and I changed form values in 2nd page to form values, but keep the ones in 4th page)
  • Components
    • Icon Button
    • Checkbox, CheckboxCard
    • TextArea
    • Modal
  • Household Info Page
    • Error handling is not done yet
  • Request Items Page

Related PRs

It's related to this PR on other pages of onboarding
I modified these pages for testing purpose, and should be overwrite by Isaiah's changes.

Anything else?

Copy link

netlify bot commented Sep 16, 2024

Deploy Preview for motherhoodbeyondbars failed. Why did it fail? →

Name Link
🔨 Latest commit e488de6
🔍 Latest deploy log https://app.netlify.com/sites/motherhoodbeyondbars/deploys/66ea0a1e4e3584000822e0b0

@parkerra parkerra self-requested a review September 16, 2024 17:40
@parkerra parkerra linked an issue Sep 16, 2024 that may be closed by this pull request
@parkerra parkerra added this to the Sprint 1 milestone Sep 16, 2024
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.

[FE] Second Half of Onboarding
2 participants