ghostguild-org/docs/LAUNCH_READINESS.md
Jennie Robinson Faber a80728f0a8 feat(signup): unify cadence UX across accept-invite, join, and account
Extract shared SignupFlowOverlay component. Static "Monthly Contribution"
label on all three contribution inputs (was misleadingly dynamic).
"Per Year"/"Per Month" toggle copy; Per Year default on accept-invite,
Per Month default on join. Live billing-summary card on both signup
flows. Welcome-heading on dashboard via ?welcome=1 for new signups.
$0-member polish on account page (hide payment-history + Solidarity
Fund prompts). State-aware contribution-change hint. Invite accept now
creates Helcim customer and sets auth cookie server-side for both free
and paid branches. Pre-registrant invite + /join signup flows manually
verified against Cleo Nguyen preReg and $0-$50 variants.
2026-04-20 12:34:59 +01:00

14 KiB
Raw Blame History

Launch Readiness

Status as of 2026-04-20. Target launch: before 2026-05-01.

Single source of truth for work that must happen before cutover. P0 blocks launch. P1 is strongly preferred but survivable. Completed items have been archived — see ~/.claude/projects/-Users-jennie-Sites-ghostguild-org/memory/project_launch_readiness_archive.md. Post-launch backlog lives in docs/TODO.md.


Current state

  • Vitest on main: 652/658 passing. 6 pre-existing failures in tests/server/api/helcim-payment.test.js — unrelated to launch-blocking work, noted in the deploy checklist for visibility.
  • main is now caught up locally (2026-04-20): feature/helcim-plan-consolidation (40 commits) and feature/contribution-amount-redesign (17 commits) fast-forwarded in. Not pushed — site is not on Netlify yet.
  • Helcim plan consolidation migration ran against prod 2026-04-18 (Monthly plan id 50302, Annual plan id 50303). Contribution-amount migration has NOT yet been run against prod.
  • Cadence/contribution UX unified across signup + edit surfaces 2026-04-20. Uncommitted in working tree — see "Cadence UX refinements" below.

Cadence UX refinements (2026-04-20, uncommitted)

Shipped across accept-invite.vue, join.vue, member/account.vue, welcome.vue, member/dashboard.vue, and a new shared SignupFlowOverlay.vue:

  • Shared SignupFlowOverlay component. Extracted from /join progress overlay; now used by both /join and /accept-invite.
  • Static "Monthly Contribution" label on all three contribution inputs (previously dynamic — flipped to "Annual Contribution" when annual cadence was selected, which was misleading because the stored value is always the monthly base).
  • "Per Year" / "Per Month" toggle copy (was "Annual" / "Monthly"). On /accept-invite, Per Year is now the default; /join stays on Per Month by default.
  • Live billing-summary card below the contribution input on both signup flows — reads e.g. "You'll be charged $180 today ($15/month × 12). Then $180 every year, until you cancel."
  • Welcome heading on dashboard for new signups: /member/dashboard?welcome=1 renders "Welcome to Ghost Guild, {name}" instead of "Welcome back, {name}". /welcome redirect now always carries the param; /accept-invite navigates to the dashboard with the param directly.
  • $0 member polish on /member/account: Payment History section hidden when contributionAmount === 0 (stops showing "first charge will appear after your next billing cycle" to members with no charges). Solidarity-Fund sentence in the Danger Zone also hidden at $0.
  • State-aware contribution-change hint on /member/account: "You'll be charged $X today to start your subscription." ($0 → paid) / "Your paid subscription will be cancelled." (paid → $0) / "Changes apply on your next billing cycle." (paid → paid, different amount).
  • Server-side invite accept now creates the Helcim customer and sets the auth cookie before returning, for both free and paid branches.

P0 — Must fix before launch

None outstanding.


P1 — Strongly preferred before launch

Charitable receipts — Phase 1 (docs/specs/receipts-launch-spec.md)

Spec exists but none of it is implemented. Phase 2 (the actual receipt generation) is post-launch (live Jan 2027), but Phase 1 must land at launch so Phase 2 has the data + compliance posture it needs. Skipping Phase 1 means: payments made between launch and Phase 2 may be missing fields needed to receipt them, and the Helcim default confirmation email may make CRA-noncompliant claims.

What needs to ship:

  • Payment logging. Every successful Helcim payment writes a record (member id, amount CAD, payment date, Helcim transaction id, payment type monthly/annual, receiptIssued: false, receiptId: null). Failed payments logged separately. No Payment model exists today; needs new model + webhook wiring in server/api/helcim/.
  • Helcim confirmation email review. Customize so it does NOT use "tax receipt," "donation receipt," or "for tax purposes," and includes the registered charity name "Baby Ghosts Studio Development Fund" plus the disclaimer wording from the spec (section 2). If Helcim's template editor can't accomplish this, suppress the Helcim confirmation and send our own via Resend.
  • Join page copy. Add the factual charity / tax-deductible note near the contribution amount input per spec section 3 — app/pages/join.vue. No form fields, just the line.
  • Member schema field. Add taxReceiptPreferences: { filesCanadianTaxes: Boolean, middleInitial: String|null, confirmedAddress: {...}|null, setupCompletedAt: Date|null } to server/models/member.js. Default null/false. Not exposed in UI yet.

Rough scope: 1 day if Helcim's email template is editable; +0.5 day if we have to route confirmations through Resend instead. Decide which path during implementation.


Deploy checklist

Applies when the site is connected to Netlify / production hosting. Nothing here is actionable until that connection exists; kept here so nothing gets forgotten at cutover.

  • Push local main to origin/main.
  • Run node scripts/migrate-contribution-amount.cjs --apply against prod Mongo BEFORE the new code serves traffic. Idempotent; dry-run on local counted 34 members. Requires MONGODB_URI in env. The script writes contributionAmount (Number) derived from existing contributionTier (String) on every Member doc; the old field is left intact for a window.
  • Set NUXT_HELCIM_MONTHLY_PLAN_ID=50302 in production env.
  • Set NUXT_HELCIM_ANNUAL_PLAN_ID=50303 in production env.
  • Decide on the 6 failing tests in tests/server/api/helcim-payment.test.js — either fix or consciously accept. Not launch-blocking, but pre-existing red tests tend to mask new regressions.

Env vars required in production (reference):

  • MONGODB_URI
  • JWT_SECRET (or NUXT_JWT_SECRET — the NUXT_ variant wins)
  • RESEND_API_KEY
  • HELCIM_API_TOKEN
  • NUXT_HELCIM_MONTHLY_PLAN_ID
  • NUXT_HELCIM_ANNUAL_PLAN_ID
  • SLACK_BOT_TOKEN
  • BASE_URL
  • OIDC_COOKIE_SECRET
  • NUXT_PUBLIC_HELCIM_PORTAL_URL

Manual browser tests still needed

Cannot be verified by Vitest. Both require a real browser + real Helcim test card + real email, via cloudflared tunnel or ngrok HTTPS (Helcim requires HTTPS for the pay.js iframe).

Shared setup (do once):

  • npx nuxi dev --https in one terminal, cloudflared tunnel --url https://localhost:3000 (or ngrok http https://localhost:3000) in another. Use the tunnel URL as BASE_URL in .env.
  • Helcim sandbox test card: see ~/.claude/projects/-Users-jennie-Sites-ghostguild-org/memory/reference_helcim_sandbox.md.
  • Apply the contribution-amount migration against local Mongo first so seeded members match the new schema:
    node scripts/migrate-contribution-amount.cjs            # dry-run
    node scripts/migrate-contribution-amount.cjs --apply    # apply
    
    After applying, confirm in mongosh: db.members.countDocuments({ contributionAmount: { $exists: true } }) should equal total member count; db.members.countDocuments({ contributionAmount: { $type: 'string' } }) must be 0.

  • Pre-registrant invite → accept flow with a paid contribution amount. Passed 2026-04-20 — both Monthly $7 and Annual $15 variants completed end-to-end. DB verified programmatically: contributionAmount stored as Number, billingCadence correct, helcimCustomerId + helcimSubscriptionId populated, status: active, no contributionTier field, preReg transitioned to accepted with memberId set.

  • Contribution-amount redesign end-to-end. Covers the full surface of the contributionTiercontributionAmount rename.

    • Signup flows — /join: Passed 2026-04-20. All 5 variants ran functionally clean (welcome-heading regression was caught, fixed via ?welcome=1 propagation through /welcome, not retested — trusted):

      1. $0 Monthly — Member created with no Helcim subscription.
      2. $5 Monthly (preset) — Helcim recurringAmount: 5.
      3. $17 Monthly (non-preset) — Helcim recurringAmount: 17, $15 chip label via findLast.
      4. $17 Annual — Helcim recurringAmount: 204, billingCadence: 'annual', Mongo stores monthly-equivalent 17.
      5. $50 Annual (top preset) — Helcim recurringAmount: 600.
    • Edit flows — /member/account as an active paid member:

      • Raise amount ($17 → $30). Confirm updateHelcimSubscription called with recurringAmount: 30 (Monthly) or 360 (Annual).
      • Lower amount ($30 → $5). Same assertion at the new values.
      • Switch cadence (Monthly $17 ↔ Annual $17). Deferred from launch. Server (update-contribution.post.js:184-189) explicitly rejects cadence changes on existing subscriptions; no UI toggle exists on /member/account. Re-scope post-launch if/when we want to support cadence switch (would need Helcim subscription replacement flow, not a plain update).
    • Admin flow — /admin/members/[id] edit:

      • contributionAmount input accepts any non-negative whole dollar. Save writes Number to Mongo.
      • No chip UI here (admin is plain number input by design).

    Assert across all flows:

    • Mongo contributionAmount is always Number, never String.
    • No contributionTier values written anywhere (greppable: db.members.findOne({}, { contributionTier: 1 }) should return whatever the migration left; no new writes to that field).
    • No "save $X", "2 months free", or discount copy appears in any UI surface. Annual is just amount × 12 exactly.
    • Guidance chip labels ($0/$5/$15/$30/$50) are matched via findLast, so $17 lands on the $15 label, $49 lands on $30, $51 lands on $50.

    Key files if debugging: app/pages/join.vue, app/pages/member/account.vue, app/pages/admin/members/[id].vue, server/api/helcim/subscription.post.js, server/api/members/update-contribution.post.js, server/api/admin/members/[id].put.js, app/config/contributions.js + server/config/contributions.js.

    Cosmetic follow-ups noted in Post-launch backlog below — won't block this test (they're naming, not behavior).


Bylaws decoupling — follow-ups (added 2026-04-18)

Context: bylaws are being amended to remove automatic termination for nonpayment. Membership status will be fully decoupled from payment status; failed payments trigger committee outreach, not status change. Copy + UI access gates already aligned in useMemberStatus.js and account.vue (2026-04-18). Server-side status gating shipped as B2 (see archive). The behavioral changes below remain.

Not blocking launch — the amendment hasn't passed yet, and the user-visible copy/UI is already consistent. Pick up once the amendment is ratified.

B1. cancel-subscription flips status to pending_payment

  • server/api/members/cancel-subscription.post.js:31,48
  • When a member cancels their paid subscription, status is set to pending_payment and contribution amount to 0. Under the new model, cancelling a payment plan moves the member to the $0 contribution — status should stay active.
  • Fix: change status: 'pending_payment'status: 'active' in both the findByIdAndUpdate payload (line 31) and the response (line 48). Comment at line 26 also needs updating ("(not cancelled) so member can re-subscribe" → reflect new framing).
  • Add coverage in tests/server/api/cancel-subscription.test.js if it doesn't already exist.

B3. Vestigial pending_payment status

  • Once payment is fully decoupled, pending_payment no longer gates anything and is functionally equivalent to active. Consider removing it from the enum (server/models/member.js:38, server/utils/schemas.js:299) and treating new signups as active from the moment of account creation.
  • Touches: signup flow (helcim/customer.post.js:34, invite/accept.post.js:48), admin filter UI (app/pages/admin/members/index.vue:45,382,499,1145, [id].vue:69,286), admin alerts (server/utils/adminAlerts.js:22,100-116, server/models/adminAlertDismissal.js:6), and a data migration to flip existing pending_payment rows to active.
  • Larger refactor — break out into its own ticket once B1 lands.

B4. Admin "Pending Payment" filter label (cosmetic)

  • app/pages/admin/members/index.vue:45,499, [id].vue:69 show pending_payment as "Pending Payment". If B3 removes the status entirely, this disappears too. If we keep pending_payment for now, rename in admin UI to "Payment setup incomplete" so admins also stop conflating it with membership state.

Post-launch backlog

See docs/TODO.md for:

  • Button minimum target size (WCAG AAA 2.5.5).
  • /oidc/interaction/[uid] routing quirk.
  • Admin layout migration from guild-* tokens to zine spec.
  • Admin dashboard quick-action button contrast.
  • Members table NAME column clipping.
  • OWASP ASVS L1 Phase 4 (file-upload validation pipeline, granular RBAC, credential encryption).
  • tickets/available.get.js:115 memberSavings block reports $0 saved for inactive members — cosmetic; suppress comparison block when !hasMemberAccess(member) if it ever surfaces in UI.

Contribution-amount redesign — cosmetic cleanup (naming only, not behavior)

  • Rename admin members column header "Tier" → "Contribution" (app/pages/admin/members/index.vue:265).
  • Delete dead app/components/TierPicker.vue.
  • Update stale tier comment in app/composables/useMemberPayment.js:59.
  • Update error log message referencing "tier" in server/api/members/update-contribution.post.js:221.
  • Rename handleUpdateTier handler in app/pages/member/account.vue.