Docs: README still marks Payments partial — Stripe card charge is implemented #73

New Issue

2026-07-18T21:03:46Z

thatguygriff commented

2026-07-18 21:03:46 +00:00

What

The README is behind the code on payments status:

README.md line 4–5: "and (coming) online payments"
README.md Implementation status table: "Payments (e-transfer/comp + receipts + HST; Stripe card charge pending)" — 🟡 Partial
README.md footnote: "Payments are deliberately deferred to the end…"

But the live Stripe card charge shipped with #7: Payment\StripeGateway creates PaymentIntents, PaymentEndpoint handles POST /payments/intent and the signature-verified POST /payments/webhook, and docs/features/payments.md already documents it as implemented.

Also stale for the same reason: the payment-seam note in docs/features/group-classes.md ("payment is deferred to #7… plugs in later") — EnrollmentEndpoint now creates payments via PaymentService.

Fix

Flip the Payments row to ✅ Implemented and drop the "(coming)" / deferral wording in README.md.
Rewrite the group-classes.md seam note to describe the wired-in payment step (keep the instructor-views follow-up note, tracked in #71).

## What The README is behind the code on payments status: - `README.md` line 4–5: "and (coming) online payments" - `README.md` Implementation status table: "Payments (e-transfer/comp + receipts + HST; **Stripe card charge pending**)" — 🟡 Partial - `README.md` footnote: "Payments are deliberately deferred to the end…" But the live Stripe card charge shipped with #7: `Payment\StripeGateway` creates PaymentIntents, `PaymentEndpoint` handles `POST /payments/intent` and the signature-verified `POST /payments/webhook`, and `docs/features/payments.md` already documents it as implemented. Also stale for the same reason: the payment-seam note in `docs/features/group-classes.md` ("payment is deferred to #7… plugs in later") — `EnrollmentEndpoint` now creates payments via `PaymentService`. ## Fix - Flip the Payments row to ✅ Implemented and drop the "(coming)" / deferral wording in `README.md`. - Rewrite the `group-classes.md` seam note to describe the wired-in payment step (keep the instructor-views follow-up note, tracked in #71).

thatguygriff referenced this issue from a commit

2026-07-18 21:07:04 +00:00

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

thatguygriff referenced a pull request that will close this issue

2026-07-18 21:07:17 +00:00

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

thatguygriff closed this issue

2026-07-18 21:08:07 +00:00

Sign in to join this conversation.