Skip to content

Add unreviewed demo services with and without invoices - #8

Merged
KCL1104 merged 1 commit into
mainfrom
codex/unreviewed-demo-services
Sep 5, 2026
Merged

KCL1104 merged 1 commit into
mainfrom
codex/unreviewed-demo-services

Conversation

@KCL1104

@KCL1104 KCL1104 commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

The demo catalog now has two additional independent service entries: C supports a Demo invoice and D does not. Both start without a Mello verification record, completing the four invoice/certification combinations alongside the existing reviewed A/B entries.

C is fulfilled by the existing Seller B endpoint and D by Seller A, using their current quotes and seller identities. Service display names distinguish the entries while the UI identifies the actual provider. Bazaar still matches the complete endpoint/payment terms, and verification remains specific to each service ID.

The API pre-deploy step registers missing C/D records and audit events idempotently, preserving existing services, reviews, policy and purchase history. The additive 20260906010000_service_display_name migration runs before API startup; Web then displays names in the survey, policy list and purchase records.

Validation: 394 API unit tests, 9 frontend tests and 58 focused database integration tests passed locally, with API type checking and repository lint. Integration coverage includes concurrent registration, preserving later edits/reviews, rollback on an ID conflict, independent certification gating and a complete mock purchase/invoice for a named unreviewed option.

@KCL1104
KCL1104 merged commit 9b3eda1 into main Sep 5, 2026
1 check passed
@KCL1104

KCL1104 commented Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Deployed merged commit 9b3eda1ffaf26bcbe138003c96127256c950e2b2 to production.

  • API 7249e8a8-d4fe-477a-9a4a-b319eeb4d78f and Web f77fe2bd-9376-4d01-b356-0e20cc3d3b49 both reached SUCCESS. The service-name migration applied and the pre-deploy registration created C/D.
  • Live read-back confirms C has a Demo invoice, D has no invoice, and both are UNREVIEWED. The A/B verification records and company policy match the pre-deployment snapshot.
  • Production browser tests passed all four requirement combinations (1 / 2 / 2 / 4 results), C selection gating, reload persistence and desktop/mobile layouts. All four test requests remain WAITING_SELECTION; purchases stayed at 10 and the settled amount stayed unchanged.
  • PR CI and merged-main CI passed.

The Railway service domain had changed to mello402.up.railway.app. WEB_PUBLIC_URL and API WEB_ORIGIN now match that domain; login and health checks passed at the new address.

@canfly1019
canfly1019 deleted the codex/unreviewed-demo-services branch September 25, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant