Skip to content
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

fix: added MS to ISiKMedikationsVerordnung.reasonCode & .reasonReference #161

Open
wants to merge 6 commits into
base: TC_4.0.1
Choose a base branch
from

Conversation

patrick-werner
Copy link
Contributor

@patrick-werner patrick-werner commented Jan 21, 2025

Contributor Pull Request

Description

  • fixes PTDATA-1280 by adding MS to ISiKMedikationsVerordnung.reasonCode & .reasonReference
  • updated dependency to de.basisprofil.r4: 1.5.1 and de.gematik.isik-basismodul: 4.0.1

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this IG / specification.
  • My change requires a change to the documentation or narrative (intend) of the IG.
  • I have already updated the documentation / narrative (intend) accordingly.

* ^short = "Grund der Medikation (codiert)"
* ^comment = " Festlegung zum MS: Die Elemente .reasonCode und .reasonReference MÜSSEN nach OR-Logik in der Ausgabe verwendet werden, d.h. nur eines MUSS geliefert werden können. Weiterhin MÜSSEN beide Elemente interpretiert werden können.

Begründung zu Must-Support: Konsolidierung mit MII."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: auf welches Profil bezieht sich das?


Begründung zu Must-Support: Konsolidierung mit MII."
* coding MS
* system 1..1 MS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue: hier fehlt die Begründung für die Kardinalitäten
comment: in diesem Profil sehe ich nur MS, aber keine Einschränkung von Kardinalitäten für system und Code : https://simplifier.net/guide/mii-ig-medikation-v2/ImplementationGuide-2.x-TechnischeImplementierung-MedicationRequest-2.x?version=current

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich füge die Motivation gerne ein. Codings sollten immer ein system und code haben. Wir werden das in die best-practices von HL7-de aufnehmen. In Stufe5 sollten wir das über alle Module überprüfen/ das ISiKCoding Profil aus dem Basismodul nutzen, und in den übergreifenden Festlegungen festhalten

* ^comment = " Festlegung zum MS: Die Elemente .reasonCode und .reasonReference MÜSSEN nach OR-Logik in der Ausgabe verwendet werden, d.h. nur eines MUSS geliefert werden können. Weiterhin MÜSSEN beide Elemente interpretiert werden können.

Begründung zu Must-Support: Konsolidierung mit MII."
* reference 1..1 MS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue: s.o.

# Conflicts:
#	Resources/fsh-generated/fsh-index.txt
#	Resources/sushi-config.yaml
#	package.json
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.

2 participants