Skip to content

Simone on fhir must support#12

Open
simoneOnFhir wants to merge 3 commits intomainfrom
simoneOnFhir-MustSupport
Open

Simone on fhir must support#12
simoneOnFhir wants to merge 3 commits intomainfrom
simoneOnFhir-MustSupport

Conversation

@simoneOnFhir
Copy link
Contributor

No description provided.

…g-mit-HL7/Übersicht/Spezifikation/Profilierung/Umgang-mit-Must-Support-Flags.page.md

Co-authored-by: Patrick Werner <pa.f.werner@gmail.com>
Copy link

Choose a reason for hiding this comment

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

Ist nicht eher eine Verpflichtung, da MS immer pro Spec definiert werden muss?

Copy link

Choose a reason for hiding this comment

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

Erzeugt das nicht einen großen Aufwand für die Spezifikateure und sollte dazu eigentlich im IG stehen und nicht in den Profilen? Auch um nicht alle Informationen in den Profilen zu haben?

Copy link
Member

Choose a reason for hiding this comment

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

Unsere Empfehlung wäre den IG automatisiert auf Basis dieser Kommentare zu generieren. In unseren Projekten hatten wir das Problem, dass ansonsten Profil und IG von einander abweichen. Wenn ein Kommentar im Profil steht und man dann das Profil ändert stolpert man eher über den Kommentar und aktualisiert diesen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wir müssen auch immer bedenken, dass viele Implementierer die Profile als die Source of Truth sehen und gerade auf Simplifier sogar oft nur die Profile anschauen, nicht den IG...
Steng genommen geht es aber in der Empfehlung hier weniger um den ORT an dem man die Begründung festhält, sondern eher, DASS man es tut.

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.

4 participants