Unsupervised/unsupervised-scheduler
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases 12 Wiki Activity

Update README and group-classes doc to reflect shipped Stripe payments #74

Merged
thatguygriff merged 1 commits from docs/readme-payments-status into main 2026-07-18 21:08:07 +00:00
Conversation 0 Commits 1 Files Changed 2
+9 -11
thatguygriff commented 2026-07-18 21:07:17 +00:00
Owner
Copy Link
Copy Source

Fixes #73

What & why

The README was behind the code on payments:

  • The intro said online payments were "(coming)".
  • The implementation-status table marked Payments 🟡 Partial with the "Stripe card charge pending", and a footnote said payments were deferred to the end.

The live Stripe card charge shipped with #7 (Payment\StripeGateway PaymentIntents, POST /payments/intent, signature-verified POST /payments/webhook), and docs/features/payments.md already documents it as implemented — so the Payments row is now ✅ and the deferral footnote is gone.

Also rewrote the stale payment-seam note in docs/features/group-classes.md ("payment is deferred to #7… plugs in later"): enrolments now create and link payments via Payment\PaymentService / EnrollmentRepository::setPaymentId. The instructor-views follow-up note stays, now pointing at #71.

Docs-only change; composer test still passes (365 tests).

🤖 Generated with Claude Code

Fixes #73 ## What & why The README was behind the code on payments: - The intro said online payments were "(coming)". - The implementation-status table marked Payments 🟡 Partial with the "Stripe card charge pending", and a footnote said payments were deferred to the end. The live Stripe card charge shipped with #7 (`Payment\StripeGateway` PaymentIntents, `POST /payments/intent`, signature-verified `POST /payments/webhook`), and `docs/features/payments.md` already documents it as implemented — so the Payments row is now ✅ and the deferral footnote is gone. Also rewrote the stale payment-seam note in `docs/features/group-classes.md` ("payment is deferred to #7… plugs in later"): enrolments now create and link payments via `Payment\PaymentService` / `EnrollmentRepository::setPaymentId`. The instructor-views follow-up note stays, now pointing at #71. Docs-only change; `composer test` still passes (365 tests). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
thatguygriff added 1 commit 2026-07-18 21:07:18 +00:00
Update README and group-classes doc to reflect shipped Stripe payments
CI / Tests (PHP 8.2) (pull_request) Successful in 38s
Details
CI / Tests (PHP 8.1) (pull_request) Successful in 44s
Details
CI / No Debug Code (pull_request) Successful in 1s
Details
CI / Coding Standards (pull_request) Successful in 2m49s
Details
CI / PHPStan (pull_request) Successful in 2m54s
Details
CI / Tests (PHP 8.3) (pull_request) Successful in 2m42s
Details
CI / Build Plugin Zip (pull_request) Skipped
Details
da9a449d55 
The README still listed Payments as partial with the Stripe card charge
pending, and group-classes.md still described the pre-#7 payment seam.
Both are behind the code: StripeGateway/PaymentEndpoint ship the live
card charge, and enrolments create and link payments via PaymentService.

Fixes #73

Co-Authored-By: Claude Fable 5 <[email protected]>
thatguygriff merged commit 05d1728248 into main 2026-07-18 21:08:07 +00:00
thatguygriff deleted branch docs/readme-payments-status 2026-07-18 21:08:07 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
 enhancement
Extends an existing feature
 feature
New user-facing capability
 payments
Billing, Stripe, receipts, reporting
 security
Security vulnerability or hardening
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
thatguygriff (James Griffin-Allwood)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Unsupervised/unsupervised-scheduler#74