backoffice-documentation-certificates-task
Scheduled task that selects monthly Best Ever Month winners from the database, generates their certificate documents and emails them to Travel Counsellors.
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 |
| Frameworks | Microsoft.Extensions.Hosting 8, FluentValidation 12, TC.Phenix.ApplicationFramework 4.7 |
| Runtimes | .NET 8 |
| Tests / CI | some / CI in place |
| Last commit | 2026-07-10 |
| Owner | Owner unknown (mapping to be curated) |
| Domain | Bookings and fulfilment (system generated; curate in repo-owners.json to override) |
| Product area | Phenix |
| Hosted | On-prem Deployed to our own machines
|
| Releases | 1 release pipeline(s): \ops\documentationcertificates\backoffice-documentation-certificates-task |
| Integration | rest, wcf |
| Source | Open in Azure DevOps |
Risk flags
Each flag links to its glossary definition.
Dependencies in context
Entanglement
kernel Bound into the core by 2 mechanism(s).
- Pinning dependency: mail-service, mailtemplate-service
- Speaks WCF (client)
Detaches when its pinning dependencies are cut or multi-targeted (mail-service, mailtemplate-service); and when its WCF surface gains a modern interface or its WCF calls move to one.
Depends on / used by
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.
- Consumes WCF contract packages: never take a breaking client bump mid-feature, and treat obsolete warnings as migration debt
- 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
Feasible now (1 owned database)The database is solely owned by this system, so the migration is self-contained. Recommended target: PostgreSQL.
- Inventory the schema and query surface of sqlserver:DocumentationCertificatesTask: 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
backoffice-documentation-certificates-task
Purpose
Console task (TC.Documentation.CertificatesTaskService) that runs on a schedule (processing day/hour from config), calls the BestEverWinners_SelectForCertificates stored procedure to find Best Ever Month commission winners, generates certificates through the documentation-builder client, and emails them using the internal mail services.
Tech stack
Single C#/.NET 8 console app with Generic Host DI, FluentValidation, Newtonsoft.Json, Microsoft.Data.SqlClient, and internal packages: TC.ConnectionBroker.* (SQL + WCF), TC.Mail.ServiceClient, TC.Mail.Template.ServiceClient, TC.DocumentationBuilder.BestEverMonthCertificate.Client, TC.Phenix.CompanyInfo. Includes a mock data-access implementation for safe test runs (FakeData flag).
Deployment
One build pipeline (backoffice-documentation-certificates-task, YAML using shared InternalOSS/azure-pipelines-templates) and one release pipeline (\ops\documentationcertificates\backoffice-documentation-certificates-task); release_count 1. Deployed as a scheduled console task (batch launcher TC.Documentation.CertificatesTaskService.bat), with per-environment appsettings files.
Dependencies
Internal: tc-connectionbroker (resolves the SQL connection string at runtime), mail-service and mailtemplate-service (sending templated certificate emails), and the certificate document generation service originating in backoffice-documentation-booking (via the TC.DocumentationBuilder.BestEverMonthCertificate.Client package). External: none.
Data stores
SQL Server accessed through the connection broker under the logical name DocumentationCertificatesTask; reads winner data via the dbo.BestEverWinners_SelectForCertificates stored procedure. No schema owned in this repo.
Integration points
REST/HTTP to internal mail and documentation-builder services; WCF client support via TC.ConnectionBroker.Wcf / System.ServiceModel.Primitives for legacy service calls. No message bus.
Code health
Dedicated xUnit-style test project covering the BEMC processor and helpers, run in CI (testSelection: **/*.Tests.csproj). Last commit 2026-07-10 — active. Small, focused codebase.
Modernisation risks
Low priority: modern .NET 8 runtime with tests and YAML CI. Minor hygiene items: personal developer email addresses hard-coded in appsettings.json, FakeData defaulting to true relies on correct per-environment overrides, and the WCF client dependency ties it to legacy endpoints.