phenix-honeymoon-api
Small .NET 8 API for loading and updating honeymoon-account details (photo, welcome/thank-you text, target amount, end date) on a Phenix booking so myTC guests can contribute towards the balance.
Health score
0 to 100 across seven weighted factors; nothing is hidden. Green is 75 and above, amber 50 to 74, red below 50. How scoring works.
Facts
| Languages | csharp, sql |
| Frameworks | ASP.NET Core 8 |
| Runtimes | .NET 8 |
| Tests / CI | some / CI in place |
| Last commit | 2026-04-10 |
| Owner | Owner unknown (mapping to be curated) |
| Domain | Public web (system generated; curate in repo-owners.json to override) |
| Product area | Phenix |
| Hosted | On-prem Deployed to our own machines
|
| Releases | 2 release pipeline(s): \ops\honeymoon\phenix-honeymoon-api; \ops\honeymoon\phenix-honeymoon-api stealth |
| Integration | rest |
| Source | Open in Azure DevOps |
Dependencies in context
Entanglement
loose No binding mechanisms apply: this system needs no untangling work.
Migration playbooks
Generated from this repo's assessment data: what is feasible, what blocks it, and the steps in order. Ratings: green means actionable now, amber means sequence other work first, red means blocked, grey means not applicable.
ZDD adoption
Ready with guardrailsClose: the deployment path exists, but specific checks are needed first, typically raising tests, auditing serialized data, or respecting a shared schema with expand and contract. Test coverage is thin; ZDD shifts the safety net onto automation.
- 1 system(s) depend on this repo; during a rolling deploy they will talk to old and new versions at once, so responses must stay compatible
- Adopt the four-stage discipline from the ZDD engineering standard: pre-deployment compatibility, deployment, feature activation behind a flag, post-release cleanup.
- Raise automated test coverage first: ZDD trades the end-of-sprint manual gate for continuous automated confidence, so the tests are the entry ticket.
- Make every change dual-version safe: backwards-compatible APIs (new major version for breaking wire changes), expand and contract for any schema change, and consumers tolerant of unknown fields.
- Audit serialized data: anything in Redis, queues or shared caches must deserialize cleanly across adjacent versions.
- Wrap behaviour changes in feature flags so code deploys dark and release becomes a business decision, not a deployment event.
- Route deployments through the Workflow Engine's automated drain, deploy, health-check, return cycle (the path proven by the first automated production run in mid-2026).
- Verify with area-scoped regression rather than the whole-platform cycle, and watch error rates after rollout with automated rollback on alert.
SQL Server exit
Blocked by a shared databasesqlserver:CustomerPortal is shared, so this system cannot change engine on its own: every sharer moves in convoy. Either agree the move with all sharers as one programme, or first split this system's data out behind an owned API, then follow the steps below.
- Inventory the schema and query surface of sqlserver:CustomerPortal: tables, stored procedures, triggers, cross-database joins and anything else engine-specific.
- Confirm the target: PostgreSQL (relational model; keep transactions and joins, swap the engine).
- Data access is not through an ORM, so port the SQL by hand: translate T-SQL constructs, replace stored procedures with application code or target-native equivalents, and add integration tests around the queries that matter first.
- Stand up the target alongside SQL Server, backfill with an ETL pass (pgloader or an export/import job), then dual-write or replicate until parity is proven.
- Move reads to the target, watch correctness and latency, then cut writes over and keep SQL Server read-only until confidence is earned.
- Decommission the SQL Server database and remove the connection strings and firewall paths that pointed at it.
Containerisation
Ready to containeriseModern runtime, no hosted legacy transport: this service can follow the estate's proven container path (Dockerfile, shared pipeline template, ECS via Terraform).
- Add a Dockerfile and make the app read all configuration from environment variables and the secrets manager, never from machine-local files.
- Log to stdout and expose a health check endpoint so ECS can supervise it.
- Adopt the shared ci.yml pipeline template with the container parameters (awsRegion, containerAfterBuild, ecsTerraformRepo), which is exactly how the estate's existing AWS services deploy.
- Define the service in the Terraform repo, deploy to a non-production environment, then shift traffic and disarm the deployment-group release pipeline.
Recommended actions
No specific actions for this repo. It either needs only normal maintenance, or its work is covered by an estate-level action. See the full programme.
Assessment notes
phenix-honeymoon-api
Purpose
Manages the honeymoon extras on a booking: a ReadController (search by Phenix id returning a public summary, load by SalesOrderId for internal use) and a WriteController (create account, patch update). Guests use the myTC website to contribute towards the booking balance quoting the reference; this API only handles the associated info (photo, welcome and thank-you text, target amount, end date), not payments.
Tech stack
C# ASP.NET Core on .NET 8, three projects (Api, Domain with raw System.Data.SqlClient data access, and a SQL script Database project with Change/Create folders) plus one xunit test project. Uses TC.ConnectionBroker (Sql/Mongo/Api), TC.Exceptions middleware, WebApi.Logger request/response logging into MongoDB, TC.Common.Hypermedia, Swashbuckle and TC health checks.
Deployment
Two release pipelines under \ops\honeymoon\ (phenix-honeymoon-api and a stealth variant, release_count 2) from one build pipeline. YAML CI (azure.pipelines.yml) builds the API and Database projects using the shared azure-pipelines-templates repo.
Dependencies
Internal packages: TC.ConnectionBroker.* (tc-connectionbroker) and TC.Exceptions.WebApiClient.* (tc-exceptions); WebApi.Logger comes from a webapi-logger repo that is not in the included estate list. No third-party SaaS beyond the Azure-hosted database.
Data stores
SQL Server database resolved via ConnectionBroker connection CustomerPortal — per the README the honeymoon data lives in Azure SQL (tcapp.database.windows.net), shared with the customer-portal domain; schema is versioned in-repo under HoneymoonAccount/Change|Create. MongoDB honeymoonapi_logs stores request/response logs.
Integration points
REST only (versioned read/write controllers with Swagger). Consumed by myTC and back-office services (backoffice-receipt-service calls it).
Code health
One xunit test project covering create/update validation only — thin coverage for an admittedly small API. Genuinely informative README (a rarity in the estate) documenting the domain and pipelines. Maintained: last commit 2026-04-10. Minor oddity: appsettings.json is UTF-16 encoded.
Modernisation risks
Low: already on .NET 8 with YAML CI and a clear, narrow purpose. Watch items are the shared CustomerPortal Azure SQL database (coupling with the customer-portal/myTC domain) and shallow test coverage of the data-access layer.