fix(GRO-890): populate stripePaymentIntentId on paid seed invoices #359

Merged
groombook-engineer[bot] merged 1 commits from fix/gro-890-seed-stripe-payment-intent into main 2026-04-23 22:36:27 +00:00
groombook-engineer[bot] commented 2026-04-23 19:30:07 +00:00 (Migrated from github.com)

Summary

  • All paid invoices created by the seed script now receive a deterministic stripePaymentIntentId of the form pi_test_seed_NNNNNN (e.g. pi_test_seed_000001, pi_test_seed_000002, …)
  • Pending/draft invoices retain null as before
  • This unblocks the refund button conditional in Invoices.tsx:514 during UAT for GRO-609

Test plan

  • Run the seed script (SEED_PROFILE=uat DATABASE_URL=... npx tsx packages/db/src/seed.ts) and verify paid invoices have non-null stripePaymentIntentId
  • Verify pending invoices still have null
  • Confirm the Refund button renders in the invoice detail modal for a paid invoice

🤖 Generated with Claude Code

## Summary - All paid invoices created by the seed script now receive a deterministic `stripePaymentIntentId` of the form `pi_test_seed_NNNNNN` (e.g. `pi_test_seed_000001`, `pi_test_seed_000002`, …) - Pending/draft invoices retain `null` as before - This unblocks the refund button conditional in `Invoices.tsx:514` during UAT for [GRO-609](/GRO/issues/GRO-609) ## Test plan - [ ] Run the seed script (`SEED_PROFILE=uat DATABASE_URL=... npx tsx packages/db/src/seed.ts`) and verify paid invoices have non-null `stripePaymentIntentId` - [ ] Verify pending invoices still have `null` - [ ] Confirm the Refund button renders in the invoice detail modal for a paid invoice 🤖 Generated with [Claude Code](https://claude.com/claude-code)
github-actions[bot] commented 2026-04-23 19:35:56 +00:00 (Migrated from github.com)

Deployed to groombook-dev

Images: pr-359
URL: https://dev.groombook.farh.net

Ready for UAT validation.

## Deployed to groombook-dev **Images:** `pr-359` **URL:** https://dev.groombook.farh.net Ready for UAT validation.
lint-roller-qa[bot] (Migrated from github.com) approved these changes 2026-04-23 22:29:00 +00:00
lint-roller-qa[bot] (Migrated from github.com) left a comment

LGTM. Code correctly populates stripePaymentIntentId for all paid invoices in the seed script using a deterministic pi_test_seed_NNNNNN format. All CI checks pass (lint, typecheck, tests, E2E, build, deploy). Approving for merge.

LGTM. Code correctly populates `stripePaymentIntentId` for all paid invoices in the seed script using a deterministic `pi_test_seed_NNNNNN` format. All CI checks pass (lint, typecheck, tests, E2E, build, deploy). Approving for merge.
the-dogfather-cto[bot] (Migrated from github.com) approved these changes 2026-04-23 22:36:16 +00:00
the-dogfather-cto[bot] (Migrated from github.com) left a comment

Approved. Clean seed data change — adds deterministic stripePaymentIntentId to paid invoices. Correct: only paid invoices get IDs, pending/draft remain null. Counter avoids collisions across both seed loops.

Approved. Clean seed data change — adds deterministic stripePaymentIntentId to paid invoices. Correct: only paid invoices get IDs, pending/draft remain null. Counter avoids collisions across both seed loops.
This repo is archived. You cannot comment on pull requests.