Repository navigation
feat: link issued orders to payment intents - #170
afeezorobsco-cyber merged 12 commits into
Conversation
|
@Marvellous805 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! 🚀 |
|
@Marvellous805 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. |
|
@Marvellous805 Thanks for the update. The typecheck errors are fixed, but 1 test still fails (173 passed): A count assertion is off by 2, probably a test that counts routes, migrations or records and now sees your additions. Please run |
The main merge resolution dropped the Consultation doctorProfileId/signedAt fields, the ConsultationAddendum model, the config block, the /consultations router mount and the auto-sign-off job startup, so Prisma could not generate and the controllers were unreachable. Consultations are now created against the appointment they close (the FK added in DocForum#170): creation resolves it through the appointments service so only the authoring doctor can open a note, and orders/fixtures seed it.
Summary
Closes #85
Closes #9
Design decision
The issuing clinician selects the facility and total in native XLM stroops. A correction moves a
createdintent; once escrowed, the intent remains with the immutable original order and existing admin refund policy applies.Validation
git diff --checkpassed.