-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add optional description id extension #624
Comments
Thanks Daniel, this looks good. |
Is there a standard for the request parameter name? The fork uses |
(Internal ticket ISTO-124 ) |
I don't think there is a standard parameter name, and something like x-include-description-id could be an option which more clearly states it's a non-standard parameter. Further, I can go ahead and create unit tests and documentation. |
That new suggested parameter name looks good to me. |
I had the need for adding description ids to ValueSet $expand result. The use case was to use FHIR $expand output as a basis for making batch description change requests to the SNOMED authoring platform. Also, there is a standard extension for description ids: https://build.fhir.org/ig/HL7/fhir-extensions//StructureDefinition-coding-sctdescid.html
There is a working branch in my fork: https://github.com/danka74/snowstorm/tree/description-ids
but would require unit tests and documentation.
The implementation adds a new parameter to the $expand operation definition "includeDescriptionId" which if it's =true adds the extension above to the FHIR expansion results.
Cheers,
Daniel
The text was updated successfully, but these errors were encountered: