3.5.0 #161

jkiddo · 2025-05-05T13:16:40Z

No description provided.

This reverts commit 946f562.

…nter

* added DkCoreDocumentReference * Update aliases.fsh * Alternative DocRef approach (#158) * Alternative DocRef approach * Further danish constraints * Removed unused alias * update scripts * Update DkCoreDocumentReference.fsh --------- Co-authored-by: Ole Vilstrup <ovi@medcom.dk>

150 profiling of encounter

Added context.sourcePatientInfo only Reference(DkCorePatient)

outcommented context.sourcePatientInfo only Reference(DkCorePatient)

Created short intro page for DocumentReference

…et expansion

Explicit display values for IEEEx73 codes no longer needed

Copilot

Pull request overview

This PR represents the 3.5.0 release of the DK Core FHIR implementation guide, introducing significant enhancements including new profiles for encounters and document references, expanded terminology support, and improved alignment with international standards.

Key Changes:

Added comprehensive Encounter profile with Danish-specific extensions for planned dates and care provider
Introduced MinimalDocumentReference profile based on IHE MHD for document sharing in Danish XDS context
Corrected NPU terminology URL from OID to canonical URL (http://npu-terminology.org)
Expanded practitioner qualifications to support multiple official health authorizations (0..1 → 0..*)
Added new value sets for encounter priorities and practice setting codes
Enhanced documentation with detailed implementation guidance for encounters and document references
Updated build scripts to include new _build.sh and _build.bat utilities

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
sushi-config.yaml	Updated special-url format to YAML array, added dependencies (hl7.fhir.extensions.r5, ihe.iti.mhd), included NPU URL
input/pagecontent/index.md	Added link to standards catalogue, corrected Danish prefix notation to [DA], added IPA compliance details (contains spelling errors)
input/pagecontent/StructureDefinition-dk-core-minimaldocumentreference-intro.md	New documentation explaining IHE MHD-based DocumentReference profile and its conformance approach (contains spelling errors)
input/pagecontent/StructureDefinition-dk-core-encounter-intro.md	New documentation detailing encounter usage, diagnosis coding, priorities, and R5 compatibility (contains spelling errors)
input/fsh/valueSets.fsh	Removed display strings from IEEE codes, added encounter priority and practice setting value sets, simplified SNOMED version reference
input/fsh/extensions.fsh	Added three new extensions: PlannedStartDate, PlannedEndDate, and CareProvider for encounters (contains spelling error)
input/fsh/codeSystems.fsh	Updated NPU code system metadata with correct URL, OID identifier, contact information, and license
input/fsh/aliases.fsh	Corrected NPU alias to use canonical URL, added new aliases for v3-ActPriority, XDS OIDs, and IANA OIDs
input/fsh/IPAProfiles.fsh	Uncommented SNOMED CT expansion parameter for includeDesignations
input/fsh/DkDocumentVersionIDExtension.fsh	New extension for DocumentReference version ID (R5 backport)
input/fsh/DkCorePractitioner.fsh	Changed qualification cardinality to allow multiple authorizations (0..1 → 0..*), updated example
input/fsh/DkCoreOrganization.fsh	Added two new example instances for hospital department and sub-department
input/fsh/DkCoreMinimalDocumentReference.fsh	New profile and two example instances for APD-DK and PLR-DK document references
input/fsh/DkCoreEncounter.fsh	New profile and two example instances demonstrating encounter usage
input/fsh/DkCoreBasicParameter.fsh	Fixed constraint from valueQuantity.system to valueQuantity.code
_updatePublisher.sh	Added download logic for new _build.sh and _build.bat scripts
_updatePublisher.bat	Increased ping timeout, added publisher location display, added _build script downloads
_build.sh	New comprehensive build script with menu-driven interface for building IGs
_build.bat	New comprehensive build script for Windows with menu-driven interface for building IGs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

input/pagecontent/index.md

input/pagecontent/StructureDefinition-dk-core-minimaldocumentreference-intro.md

input/fsh/extensions.fsh

input/pagecontent/index.md

input/fsh/valueSets.fsh

Copilot · 2025-11-24T17:21:41Z

input/fsh/DkCorePractitioner.fsh

  * ^slicing.discriminator.path = "identifier.system"
  * ^slicing.rules = #open
  * ^slicing.ordered = false
  * ^slicing.description = "Slice based on the identifier.systemm value which allows for official qualifications to\nDeclaring the slices, and their cardinalities, to allow a KL-code and a SNOMED CT code"


The cardinality of the officialHealthAuthorization slice has been changed from 0..1 to 0..*, allowing practitioners to have multiple official health authorizations. This is a breaking change that increases flexibility but could impact validation of existing data. The updated example correctly demonstrates this by showing a practitioner with two qualifications. Ensure that all consumers of this profile are prepared to handle multiple authorizations.

Suggested change

* ^slicing.description = "Slice based on the identifier.systemm value which allows for official qualifications to\nDeclaring the slices, and their cardinalities, to allow a KL-code and a SNOMED CT code"

* ^slicing.description = "Slice based on the identifier.system value. The cardinality of the officialHealthAuthorization slice has been changed from 0..1 to 0..*, allowing practitioners to have multiple official health authorizations. This is a breaking change; ensure all consumers are prepared to handle multiple authorizations."

// BREAKING CHANGE: The cardinality of officialHealthAuthorization is now 0..*, allowing multiple authorizations per practitioner. Update all consumers accordingly.

We're okay with that in a core IG. Such constraints can be made downstream

@jkiddo Note that Copilot actually found and fixed a spelling mistake in the change suggestion above that you missed: systemm -> system

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…eference-intro.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 8 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

input/pagecontent/index.md

Copilot · 2025-11-24T17:42:09Z

input/fsh/codeSystems.fsh

+* ^contact.telecom[=].value = "labterm@sundhedsdata.dk"
 * ^experimental = false
-* ^caseSensitive = false
+* ^caseSensitive = true


The caseSensitive property for the NPU CodeSystem is being changed from false to true. This is a breaking change that could affect existing implementations. NPU codes should be validated to ensure they are consistently cased in all existing profiles, examples, and value sets. Please document this breaking change in the release notes.

input/fsh/valueSets.fsh

input/pagecontent/StructureDefinition-dk-core-minimaldocumentreference-intro.md

input/fsh/DkCoreMinimalDocumentReference.fsh

input/fsh/DkCoreEncounter.fsh

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot · 2025-11-24T17:49:03Z

@jkiddo I've opened a new pull request, #194, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

jacand · 2025-11-27T12:35:31Z

input/fsh/DkCorePractitioner.fsh

  * ^slicing.discriminator.path = "identifier.system"
  * ^slicing.rules = #open
  * ^slicing.ordered = false
  * ^slicing.description = "Slice based on the identifier.systemm value which allows for official qualifications to\nDeclaring the slices, and their cardinalities, to allow a KL-code and a SNOMED CT code"


@jkiddo Note that Copilot actually found and fixed a spelling mistake in the change suggestion above that you missed: systemm -> system

input/pagecontent/index.md

input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

Clarify the role of the implementation guide in relation to the FHIR standard and the IPA specification.

jkiddo and others added 15 commits March 23, 2025 18:09

Added DocRef based on IHE with backport extension

946f562

Revert "Added DocRef based on IHE with backport extension"

accefe1

This reverts commit 946f562.

[(150-profiling-of-encounter)] First initial draft profiling of Encou…

c8413dc

…nter

[(150-profiling-of-encounter)] First initial draft profiling of Encou…

aabac8d

…nter

[(150-profiling-of-encounter)] First initial draft profiling of Encou…

92cc5d3

…nter

[(150-profiling-of-encounter)] Ready for draft review.

904afb2

[(150-profiling-of-encounter)] Ready for draft review.

17df578

[(150-profiling-of-encounter)] Ready for pull request.

3822fd7

[(150-profiling-of-encounter)] Ready for draft review.

0b482c7

[(150-profiling-of-encounter)] Merge conflict from 3.5.0 handled

1fe956f

[(150-profiling-of-encounter)] Changes after first review

eaa537b

[(150-profiling-of-encounter)] Changes after first review

fe6447f

Merge pull request #164 from hl7dk/150-profiling-of-encounter

e2ac362

150 profiling of encounter

Update DkCoreDocumentReference.fsh

0fd43cc

Added context.sourcePatientInfo only Reference(DkCorePatient)

ovi-medcom force-pushed the 3.5.0 branch from 10ce8eb to 0fd43cc Compare August 25, 2025 10:17

ovi-medcom and others added 7 commits August 25, 2025 12:26

Update DkCoreDocumentReference.fsh

9aa27af

outcommented context.sourcePatientInfo only Reference(DkCorePatient)

Update DkCoreDocumentReference.fsh

c05e410

Create StructureDefinition-dk-core-documentreference-intro.md

ec3d4e4

Created short intro page for DocumentReference

Update StructureDefinition-dk-core-documentreference-intro.md

699bbb3

Issue 155, extra info around IPA use cases

fd993ad

Remove descriptions from SCT codes in valueSets.fsh (#167)

b3e7a0e

Explicit display values for IEEEx73 codes no longer needed for values…

97d003d

…et expansion

jkiddo mentioned this pull request Sep 26, 2025

Removing display values #168

Closed

jkiddo and others added 3 commits September 30, 2025 12:06

Uncomment includeDesignations parameter in IPAProfiles

26760ef

Merge pull request #170 from hl7dk/168-removing-display-values

1bafa3b

Explicit display values for IEEEx73 codes no longer needed

Update SNOMED CT version URL in valueSets.fsh

733e0e1

This was referenced Oct 6, 2025

Profiling of DocumentReference #149

Open

Add extra information around the use of IPA vs regular core profiles #155

Closed

Fix for #159

13bfc8a

Copilot started reviewing on behalf of jkiddo November 24, 2025 17:17 View session

Copilot finished reviewing on behalf of jkiddo November 24, 2025 17:19

jkiddo marked this pull request as draft November 24, 2025 17:21

jkiddo requested a review from j3h4 November 24, 2025 17:21

Copilot AI reviewed Nov 24, 2025

View reviewed changes

jkiddo and others added 5 commits November 24, 2025 18:28

Update input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

d7ef306

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

7358fd9

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update input/pagecontent/index.md

2e4d54d

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update input/pagecontent/StructureDefinition-dk-core-minimaldocumentr…

1d64108

…eference-intro.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update input/fsh/extensions.fsh

0f54ac3

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

jkiddo requested a review from Copilot November 24, 2025 17:36

Copilot started reviewing on behalf of jkiddo November 24, 2025 17:36 View session

Copilot finished reviewing on behalf of jkiddo November 24, 2025 17:40

Copilot AI reviewed Nov 24, 2025

View reviewed changes

jkiddo and others added 2 commits November 24, 2025 18:47

Update input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

a9a4b24

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update input/pagecontent/StructureDefinition-dk-core-encounter-intro.md

e179710

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI mentioned this pull request Nov 24, 2025

Fix typo: "imlementation" → "implementation" in index.md #194

Closed

jkiddo added 2 commits November 24, 2025 19:51

fix spelling

15b67d7

fixed spelling

599c713

jkiddo requested a review from Copilot November 24, 2025 18:53

Copilot started reviewing on behalf of jkiddo November 24, 2025 18:54 View session

Copilot finished reviewing on behalf of jkiddo November 24, 2025 18:58

Copilot AI reviewed Nov 24, 2025

View reviewed changes

Merge branch 'master' into 3.5.0

d71e6ee

jacand reviewed Nov 27, 2025

View reviewed changes

jkiddo added 3 commits November 28, 2025 08:45

Update StructureDefinition-dk-core-encounter-intro.md

a591aca

Refine description of FHIR standard and IPA specification

1299589

Clarify the role of the implementation guide in relation to the FHIR standard and the IPA specification.

Fix typo in slicing description for qualifications

c50195b

jacand self-requested a review November 28, 2025 17:08

	* ^slicing.description = "Slice based on the identifier.systemm value which allows for official qualifications to\nDeclaring the slices, and their cardinalities, to allow a KL-code and a SNOMED CT code"
	* ^slicing.description = "Slice based on the identifier.system value. The cardinality of the officialHealthAuthorization slice has been changed from 0..1 to 0..*, allowing practitioners to have multiple official health authorizations. This is a breaking change; ensure all consumers are prepared to handle multiple authorizations."
	// BREAKING CHANGE: The cardinality of officialHealthAuthorization is now 0..*, allowing multiple authorizations per practitioner. Update all consumers accordingly.

3.5.0 #161

Are you sure you want to change the base?

3.5.0 #161

Uh oh!

Conversation

jkiddo commented May 5, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

Uh oh!

jkiddo Nov 24, 2025

Choose a reason for hiding this comment

Uh oh!

jacand Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

jkiddo Nov 28, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 24, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI commented Nov 24, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

jacand Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants