enhancement: reference MS #133
Conversation
relative und absolute URLs
MaxMTheilig
requested review from
MaxMTheilig
and removed request for
MaxMTheilig
Resources/input/fsh/ISiKTermin.fsh
Outdated
| @@ -19,6 +19,8 @@ Id: ISiKTermin | |||
| * start 1..1 MS | |||
| * end 1..1 MS | |||
| * slot 0..* MS | |||
| * reference 1.. MS | |||
| * slot ^comment = "Zur Referenzierung auf eine Slot-Ressource MUSS eine Reference.reference mit einer URL verwendet werden. Bei verteilten Systemen sollten absolute URLs benutzt werden. Auf dem gleichen Host sind auch kürzere, relative URLs zulässig." //Zur Begründung: verschiedene Referenzierungs-Arten (z.B. mit Business-Identifiern) sind ggf. nicht interoperabel | |||
There was a problem hiding this comment.
Das Termin-Repository muss so gestaltet sein, dass aus Perspektive des Clients es nur eine Service-BaseUrl gibt. Im Falle eines verteilten Systems muss das Termin-Repository diese Komplexität verstecken und alles intern weiterleiten.
There was a problem hiding this comment.
Ist das dein Vorschlag für den Comment zum .slot? ... ich würde zumindest den Satz "Zur Referenzierung auf eine Slot-Ressource MUSS eine Reference.reference mit einer URL verwendet werden." vorangestellt stehenlassen... oder?
There was a problem hiding this comment.
Nein, kein Vorschlag nur eine Erläuterung. Ja, "Zur Referenzierung auf eine Slot-Ressource MUSS eine Reference.reference mit einer URL verwendet werden." können wir gern übernehmen.
There was a problem hiding this comment.
Vorschlag zur Umsetzung des Kommentars "Zur Referenzierung auf eine Slot-Ressource MUSS eine Reference.reference mit einer URL verwendet werden. Das Termin-Repository muss so gestaltet sein, dass es aus Perspektive des Clients nur eine Service-BaseUrl gibt."
- zusätzlich muss die Klarstellung zur Client-Sicht in der Basis textuell (informativ) festgehalten werden. Ggf hier als erstes: https://simplifier.net/guide/Implementierungsleitfaden-ISiK-Basismodul-Stufe-3/markdown-UebergreifendeFestlegungen-UebergreifendeFestlegungen-Rest?version=current
| @@ -19,6 +19,8 @@ Id: ISiKTermin | |||
| * start 1..1 MS | |||
| * end 1..1 MS | |||
| * slot 0..* MS | |||
| * reference 1.. MS | |||
There was a problem hiding this comment.
Das gleiche bitte auch noch einmal für Slot.schedule
|
Für das TC spricht, dass der TExtuelle Teil als MUSS-Anforderung dies impliziert. D.h. es wird nur im Profil technisch corrected ... TC! |
|
@f-peverali Ich würde vorschlagen den PR zu aufzusetzen, sodass nur die Änderung die wir haben wollen im PR drin sind. Die Änderungen sehen ansonsten gut aus. |
was wäre hier ein eleganter weg mit git? Ggf. erstmal TC_3.0.1 schließen und dann noch diesen PR direkt in den main-Stufe-3 |
|
Ich glaube mit einem git rebase sollte sich das machen lassen |
URLs update reference now MS * Update ISiKTermin.fsh: update Appointment.slot reference * Update ISiKTerminBlock.fsh: update Slot.schedule reference --------- Co-authored-by: Max Theilig <max.theilig@gematik.de>
Description
Added Reference.reference as 1.. and MS to Slot. This element must be used when referencing
Motivation and Context
Reference-mode should be constrained
PTData-495 see 2)
How has this been tested?
compilation was tested locally
Snippets or Screenshots (if necessary):
Types of changes
Checklist:
TODO