Skip to content

VED-789: Gen Validation Schema#916

Merged
Akol125 merged 108 commits intostaging/VED-508-validation-refactor-and-dq-featurefrom
VED-789-Schema-Validation
Nov 11, 2025
Merged

VED-789: Gen Validation Schema#916
Akol125 merged 108 commits intostaging/VED-508-validation-refactor-and-dq-featurefrom
VED-789-Schema-Validation

Conversation

@Akol125
Copy link
Contributor

@Akol125 Akol125 commented Oct 20, 2025

Summary

  • ✨ New Feature

This marks the beginning of a new generic validation engine that is fully configurable through a Redis-based schema. The schema defines the metadata and expression rules required for the engine to accurately parse and validate both Batch and API resources in a dynamic, extensible manner.

The validation process leverages these schemas to apply field-level and rule-based checks, ensuring flexibility without requiring code changes when validation logic evolves.

In addition, the engine features an extensive error classification system, categorising issues by severity and category. Validation results are generated in two distinct report formats, one of which is automatically exported to an S3 bucket, providing structured data for the Data Quality Team to analyse and monitor data integrity across ingestion workflows.
Read more using docs here - https://nhsd-confluence.digital.nhs.uk/pages/viewpage.action?pageId=914071143&spaceKey=Vacc&title=Immunisation%2BFHIR%2BAPI%2B-%2BGeneric%2BValidation%2BCSV%2BFHIR

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

  • I have reviewed the changes in this PR and they fill all or part of the acceptance criteria of the ticket, and the code is in a mergeable state.
  • If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
  • I have ensured the changelog has been updated by the submitter, if necessary.

@Akol125 Akol125 changed the base branch from master to staging/VED-508-validation-refactor-and-dq-feature November 10, 2025 21:29
@Akol125 Akol125 had a problem deploying to internal-dev-sandbox November 10, 2025 21:31 — with GitHub Actions Failure
@sonarqubecloud
Copy link

@Akol125 Akol125 temporarily deployed to internal-dev-sandbox November 11, 2025 11:51 — with GitHub Actions Inactive
@Akol125 Akol125 merged commit 05db712 into staging/VED-508-validation-refactor-and-dq-feature Nov 11, 2025
17 checks passed
@Akol125 Akol125 deleted the VED-789-Schema-Validation branch November 11, 2025 12:31
@JamesW1-NHS JamesW1-NHS requested review from JamesW1-NHS and removed request for JamesW1-NHS November 13, 2025 09:39
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.

4 participants