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

Convert Appointment Form into a Multi Step Form without a Stepper #96

Merged
merged 6 commits into from
Aug 26, 2023

Conversation

5hraddha
Copy link
Collaborator

Description

After following shadcn-ui/ui#318 for a while, thought it would be better to at least have a working multistep form as we are opening the platform for testing. Once we've the Stepper component ready, we can incorporate that.

As part of this PR, I've:

  • converted the appointment form to a multistep form.
  • updated the secondary footer styling
  • updated the appointment request page header

Related Issue

Partially covers #95

Acceptance Criteria

Right now, the appointment for is just one long form that seems to ask a lot for users. It would appear less overwhelming to users if the form can be split it up in to 2-3 steps.

Type of Changes

Type
🐛 Bug fix
✨ New feature
🔨 Refactoring
💯 Add tests
🔗 Update dependencies
📜 Docs

Updates

Before

image

After

image image

Testing Steps / QA Criteria

Run npm install before running the code.

@NuriyaAkh
Copy link
Collaborator

Looks beautiful, Shraddha! Good job in converting the appointment form to a multistep form

Copy link
Collaborator

@Amanda2900 Amanda2900 left a comment

Choose a reason for hiding this comment

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

Looks great @5hraddha ! Excellent job! The form still works, my data gets submitted, and filling out the form is a much more pleasant experience :)

@Amanda2900 Amanda2900 merged commit 99b9e00 into main Aug 26, 2023
1 check passed
@5hraddha 5hraddha deleted the shraddha/feature/appointment-form branch September 22, 2023 21:03
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.

3 participants