Skip to content

VED-318: Multiple entries of System - http://snomed.info/sct is not allowing for SITE & ROUTE but allowing for VACCINECODE and EXTENSION#1405

Merged
Thomas-Boyle merged 10 commits intomasterfrom
ved-318-multiple-entries-system-site-route
Apr 22, 2026
Merged

VED-318: Multiple entries of System - http://snomed.info/sct is not allowing for SITE & ROUTE but allowing for VACCINECODE and EXTENSION#1405
Thomas-Boyle merged 10 commits intomasterfrom
ved-318-multiple-entries-system-site-route

Conversation

@Thomas-Boyle
Copy link
Copy Markdown
Contributor

  • API create and update now keep only the first SNOMED coding received for site and route before shared validation runs. This delivers the required first code wins behavior for API requests without changing batch processing rules.

  • Added focused regression coverage for API create/update handling and a batch guard to confirm duplicate SITE codings are still rejected.The backend and recordforwarder service test suites both pass after the refactor and code reduction.

@github-actions
Copy link
Copy Markdown
Contributor

This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket:

VED-318

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines:
Successfully started running 1 pipeline(s).

@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 10:46 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 10:47 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle temporarily deployed to internal-dev-sandbox April 15, 2026 10:47 — with GitHub Actions Inactive
@Thomas-Boyle Thomas-Boyle enabled auto-merge (squash) April 15, 2026 12:12
@Thomas-Boyle Thomas-Boyle added bugfix This PR contains a fix for existing functionality. python Pull requests that update python code labels Apr 16, 2026
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This PR contains a fix for existing functionality. python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants