Skip to content

Add the return key attributes as specified in the conformance doc#55

Open
steventux wants to merge 1 commit intomainfrom
fix/provide-return-key-attributes-defined-in-conformance-doc
Open

Add the return key attributes as specified in the conformance doc#55
steventux wants to merge 1 commit intomainfrom
fix/provide-return-key-attributes-defined-in-conformance-doc

Conversation

@steventux
Copy link
Contributor

@steventux steventux commented Mar 19, 2026

Description

A C-FIND request can specify which fields the SCP SOP instances in response should contain
The documentation says non-requested keys will be ignored, so we are safe to always return the full set for each item.

image

(there's a whole other page of data in the table above) see the conformance doc to check this PR.

Jira link

Part of https://nhsd-jira.digital.nhs.uk/browse/DTOSS-12471

Review notes

Review checklist

  • Check database queries are correctly scoped to current_provider

The documentation says non-requested keys will be ignored, so we are safe to always return the full set for each item.
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