Repository navigation
Implement cancellation policy (issue #56), waitlist with time-boxed offers (issue #55), and reschedule (issue #54) - #164
Conversation
…-boxed offers (issue DocForum#55), and reschedule (issue DocForum#54) - Add late_cancelled status to AppointmentStatus enum - Implement cancelAppointment with 2-hour cutoff logic - Implement rescheduleSlot atomically with new slot claiming and old slot release - Implement waitlist join/leave/list/accept functionality - Add WaitlistEntry and WindowTime models to Prisma schema - Add route handlers for all new endpoints - Add validation schemas for all new requests - Fix auth repository to include timeZone field
|
@Blessed-Femi Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
@Blessed-Femi Thanks for the PR. It can't be merged yet. To unblock it: CI is failing. Here's what broke:
Please fix these errors and run the same steps locally before pushing ( Merge conflicts: this branch conflicts with Push to this same branch and CI will re-run. Once it's green and conflict-free, it'll be reviewed for merge. |
|
@Blessed-Femi Thanks for the update. The malware scan passed, but the backend CI fails at Add the back-relations, |
…-boxed offers (issue DocForum#55), and reschedule (issue DocForum#54) - Add late_cancelled status to AppointmentStatus enum - Implement cancelAppointment with 2-hour cutoff logic - Implement rescheduleSlot atomically with new slot claiming and old slot release - Implement waitlist join/leave/list/accept functionality - Add WaitlistEntry and WindowTime models to Prisma schema - Add route handlers for all new endpoints - Add validation schemas for all new requests - Fix auth repository to include timeZone field
|
@Blessed-Femi This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-core.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared files like Automated message. It won't repeat unless the PR conflicts again after your next push. |
|
@Blessed-Femi CI is failing on this branch, so it can't be merged yet. The errors from the latest run: Backend (typecheck + test + build) The new |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
@Blessed-Femi This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-core.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared files like Automated message. It won't repeat unless the PR conflicts again after your next push. |
|
@Blessed-Femi CI is failing on this branch against the latest Backend (typecheck + test + build)
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
caaa467 to
55dd27b
Compare
|
@Blessed-Femi This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-core.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared docs like Automated message. It won't repeat unless the PR conflicts again after your next push. |
|
@Blessed-Femi CI is failing on this branch against the latest Backend (typecheck + test + build)
|
|
@Blessed-Femi This PR now has merge conflicts with git fetch upstream # upstream = https://github.com/DocForum/docforum-core.git
git rebase upstream/main
# resolve the conflicts, re-run the tests, then:
git push --force-with-leaseThe usual culprits are shared docs like Automated message. It won't repeat unless the PR conflicts again after your next push. |
|
@Blessed-Femi CI is failing on this branch against the latest Backend (typecheck + test + build)
|
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
afeezorobsco-cyber
left a comment
There was a problem hiding this comment.
Sweep: malware scan clean, CI green.
|
@Blessed-Femi CI on this PR is green, but it is 20 commits behind |
|
@Blessed-Femi Thanks for rebasing. One typecheck error is left. It comes from the new test factories that Your PR makes |
The waitlist/timezone migration made DoctorProfile.timeZone required with no DB default, while the factory takes Partial overrides, so an omitted timeZone produced undefined and broke typecheck after rebasing DocForum#208. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
afeezorobsco-cyber
left a comment
There was a problem hiding this comment.
Sweep: malware scan clean, CI green.
PR resolves and closes all 4 issues: closes #54, closes #55, closes #56, closes #57. Implements cancellation policy (issue #56), waitlist with time-boxed offers (issue #57), reschedule endpoint (issue #55), and related fixes.