-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: availability page functionality #85
Conversation
Note: Use some byte (0, 1) string in database instead of true, false |
note: add in guest stuff here, move group availability to followup |
83a8b14
to
45d0aec
Compare
I think the deployment doesn't work because it's not up to date with what's on |
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.
Looks good to me. I'll go merge this in now to unblock other issues.
…ailability-page-functionality
@KevinWu098 Merging to main is blocked because of failing deployment. It might be because of the schema revert in 89a751c. Can you change |
@MinhxNguyen7 Requesting review since I made a commit on this branch |
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.
Your merge looks good to me @seancfong. That's the extent of my review.
🎉 |
Summary
Testing Plan
databaseUtils.server.ts
) -- feel free to test dates/times other than the one I listed, but some values are hardcoded (start and end times), so ymmv/auth
and create a user/availability/[YOUR_MEETING_ID_FROM_ABOVE]
Login Modal Flow
Authed User
Guest User
availabilityStore.ts
and setting the value ofguestSession.guestName
to whatever name you choose aboveFuture Followup