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

Feature/ptdata 773 anforderungen anpassen basis ressourcen #163

Merged

Conversation

f-peverali
Copy link
Contributor

löst PTDATA-773

<!--* [Person im Gesundheitsberuf](https://simplifier.net/guide/Implementierungsleitfaden-ISiK-Basismodul-Stufe-3/markdown-Datenobjekte-Datenobjekte-PersonImGesundheitsberuf?version=current)-->

Die Verwendung in diesem Modul bedeutet:
Wenn dieses Modul auf Datenobjekte aus dem Basismodul referenziert, dann MÜSSEN die referenzierten FHIR-Ressourcen konform zu den Vorgaben aus dem Basismodul sein.
Copy link
Contributor

Choose a reason for hiding this comment

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

ich fände es anschaulicher, wenn wir die Hinweise auf das erforderliche targetProfile für die Implementierung / Bestätigung textuell in den jeweiligen Referenzen der Datenobjekte ergänzen würden, ansonsten behauptet z.B. die Beschreibung von DocumentReference.subject unter https://simplifier.net/guide/Implementierungsleitfaden-ISiK-Modul-Dokumentenaustausch-Stufe-3/ImplementationGuide-markdown-Datenobjekte-DocumentReference?version=current genau das Gegenteil ("siehe FHIR-Kernspezifikation" und nicht siehe ISIKPatient...)

Copy link
Contributor

Choose a reason for hiding this comment

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

ich meine, wenn ich mir das Profil ISiKDokumentenMetadaten anschaue und unter subject - Reference(Patient) und unter der Beschreibung - "Siehe Beschreibung in der FHIR Kernspezifikation" vorfinde, dann klingt es so, als ob hier jede Patient-Resource zugelassen wäre, was im ISIK-Umfeld aber wie besprochen nicht der Fall ist. Ich finde, ein zusätzlicher textueller Hinweis ist an dieser Stelle sinnvoll

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah ok. Ja, guter Hinweis.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hier umgesetzt: c24d5ec

@alexzautke
Copy link
Contributor

Wo siehst du dort etwas Gegenteiliges?

@f-peverali f-peverali merged commit 4397350 into TC_3.0.1 Dec 11, 2023
@f-peverali f-peverali deleted the feature/PTDATA-773-anforderungen-anpassen-Basis-Ressourcen branch December 11, 2023 13:41
f-peverali added a commit that referenced this pull request Jan 5, 2024
* feat: Trigger on every push/pull

* feat: Update rulesets

* fix: FIx warnings

* Commit from GitHub Actions (CI (FHIR Validation))

* Update sushi-config.yaml

* improve requirements ressources Basismodul - ptdata-773 (#163)

* update general usage of Basis ressources

* rm: SEARCH Encounter.account.identifier + typofix

* rm: example encounter.account.identifier

* add clarification on Encounter + rm example

* rm: CpS statements for Encounter+Patient

* add requirement in profile

* comments on references to ISIK basis resources

* add notes

---------

Co-authored-by: alexey-tschudnowsky <cole1985@gmail.com>

* update dependency Basismodul

* rm redundant requirement

* rm all interaction on Encounter + Patient  in CpS

* Update ImplementationGuide/markdown/DecisionLog.md

* Commit from GitHub Actions (CI (FHIR Validation))

* fix: Revert date change

* Commit from GitHub Actions (CI (FHIR Validation))

* feat: Bump versions

* Commit from GitHub Actions (CI (FHIR Validation))

* update releasenotes

* update version and update script

* Feature/add binary page (#168)

* add binary page

* fix page context

* fix tpo

* fix typo

---------

Co-authored-by: Yannick Börner <ylboerner@outlook.com>
Co-authored-by: ylboerner <ylboerner@users.noreply.github.com>
Co-authored-by: alexey-tschudnowsky <cole1985@gmail.com>
Co-authored-by: Alexander Zautke <alexander@fire.ly>
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.

None yet

3 participants