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

ANFISK-241-Encounter.location und ANFISK-242 #407

Merged
merged 4 commits into from
Jun 7, 2024

Conversation

jschoen-gefyra
Copy link
Contributor

added Encounter.location.status to slicing and MS
added status to example

Please go the the Preview tab and select the appropriate sub-template:

Contributor PR

Version Upgrade (for Admins)

Pull Request Short Cut

Copy link
Contributor

@f-peverali f-peverali left a comment

Choose a reason for hiding this comment

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

todo: Releasenotes einpflegen

suggestion (non-blocking) : ich würde die Entscheidung zu Dekumententationszwcken entsprechend in einem comment unter Encounter unterbringen. Sinngemäß: "Anhand der hier vorgegebenen Slices SOLL keine Historie der räumlichen Unterbringung vorgehalten werden."

@alexzautke
Copy link
Contributor

@f-peverali Meinst du "Anhand der hier vorgegebenen Slices ist es nicht verpflichtend eine Historie der räumlichen Unterbringung vorzuhalten."? Andernfalls klingt es so als würden wir davon abraten.

@f-peverali
Copy link
Contributor

@f-peverali Meinst du "Anhand der hier vorgegebenen Slices ist es nicht verpflichtend eine Historie der räumlichen Unterbringung vorzuhalten."? Andernfalls klingt es so als würden wir davon abraten.
@alexzautke - Durch die "vorgegebenen Slices", die in diesem PR alle #active im Status sind, kann keine Historie abgebildet werden, richtig? Damit wäre auch dieser Formulierungsvorschlag nicht optimal.
Vielleicht dann noch etwas hinzu, dass zur Nutzung eines anderen Slices rät, "falls die Abbildung einer Historie der räumlichen Unterbringung gewünscht ist" ?

@alexzautke
Copy link
Contributor

In jedem Slice gäbe es dass .period-Element, wir haben nur kein MS-Flag darauf definiert. Ein System könnte es aber eigenständig verwenden.

@f-peverali f-peverali changed the title ANFISK-241-Encounter.location ANFISK-241-Encounter.location und ANFISK-242 May 29, 2024
@f-peverali
Copy link
Contributor

f-peverali commented May 29, 2024

In jedem Slice gäbe es dass .period-Element, wir haben nur kein MS-Flag darauf definiert. Ein System könnte es aber eigenständig verwenden.

verstehe den hinweis nicht... in den Slice ist immer #active , was hilft da die period zur Abbildung? Wenn ein Slice #active ist, soll es ja gerade NICHT die Abbildung der Ortshistorie unterstützten, oder?

@alexzautke
Copy link
Contributor

Ah, ich hatte nicht auf dem Schirm, dass wir status = active im Profil verlanden. Slicing auf PhysicalType und Status hätte doch auch gereicht? Passt. Weiterhin belieb ich dabei, dass wir sagen sollten, dass es nicht notwendig ist die Historie vorzuhalten. Ein Hersteller könnte weiterhin einen anderen Status freiwillig implementieren. Wir würden ja nicht davon abraten.

@@ -672,6 +699,12 @@
"min": 1,
"mustSupport": true
},
{
"id": "Encounter.location:Station.status",
Copy link
Contributor

Choose a reason for hiding this comment

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

issue: pattern muss auf location , nicht wie hier auf status

@f-peverali f-peverali merged commit 826d5ea into rc/main-stufe-4 Jun 7, 2024
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.

5 participants