-
Notifications
You must be signed in to change notification settings - Fork 16
Implement bookers page overhaul #1407
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
Conversation
devmount
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: Andreas <[email protected]>
eb1568e to
0dc3f81
Compare
|
@devmount thank you so much for the review! Great catch, I've asked the Design team on Matrix and, as you suggested, they decided to leave the Applied the change and flipping it to ready for review! |
devmount
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thank you Davi!
|
The new booking page looks awesome! 🥇 |
|
Thank you so much @rwood-moz ! |

Description of the Change
Refactored the styling of the Booker's Page / Booking Page according to the Zeplin file:
https://app.zeplin.io/project/66dd0f2b2fe8f1b63cda36f1/screen/68cd9bb8f41b5541ea14f7ad
Note
If you Availability settings has
Automatically confirm bookings if time is availableset, you will see an extra helper text under theBook AppointmentbuttonScreenshots
Light mode / Empty state

Light mode / Event selected

Dark mode / Empty state

Dark mode / Event selected

Benefits
Known issues / Things to improve
schedulestable in the DB calledlocation_typewhich is defaulting toinpersoninstead ofonline. Since we don't have the option to change this in the UI anywhere yet, we can't rely on this field to hide / show the related text so the "Virtual" text is hardcoded as of now.Applicable Issues
Fixes #1285
Fixes #1385