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

Encounter Beschreibung für Terminplanung #87

Merged
merged 12 commits into from
May 17, 2023

Conversation

f-peverali
Copy link
Contributor

@f-peverali f-peverali commented May 4, 2023

Für den IG fehlt eine Beschreibung zum Encounter im Kontext der Terminplanung. Diese sollte hiermit erstellt und mit den Stakeholdern abgestimmt werden - dazu auch die Kommentare in #60 beachten.

@f-peverali f-peverali changed the title Encounter Beschreibung für Terminplnaung Encounter Beschreibung für Terminplanung May 4, 2023
@f-peverali f-peverali changed the base branch from main-Stufe-3 to 3.0.0-rc2 May 11, 2023 13:13
@f-peverali f-peverali marked this pull request as ready for review May 11, 2023 13:28
Copy link
Contributor

@alexzautke alexzautke left a comment

Choose a reason for hiding this comment

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

Kleine Anmerkung, ansonsten sieht es gut aus

Copy link
Contributor

@jcaumann jcaumann left a comment

Choose a reason for hiding this comment

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

bis auf den benannte editorischen Änderungsvorschlag für mich gut

Copy link
Contributor

@jcaumann jcaumann left a comment

Choose a reason for hiding this comment

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

doch noch 2 weitere editorische Anmerkungen zu "alten" Texten:

  • auf der Seite "Interaktionen" funktionieren auf der gerenderten Seite die Anchor auf die Seite mit den Operationen nicht. Die Links landen immer oben auf der Seite und nicht bei dem referenzierten Kapitel.
  • Auf der Seite zu ISiKNachricht taucht in den Spiegelstrichen oben auf der Seite 2x "soll" in Kleinschrift auf. Das MUSS eigentlich auch groß geschrieben werden ("SOLL").

@f-peverali
Copy link
Contributor Author

  • auf der Seite "Interaktionen" funktionieren auf der gerenderten Seite die Anchor auf die Seite mit den Operationen nicht.

Soweit ich sehe wird nur die Seite als ganze referenziert. Wir haben zu dem Thema Referenzierung von Seitenabschnitten bereits ein Ticket. Sollten das für alle Projekte umsetzen. Aber für die Kommentierung keine Priorität, da keine normative Änderung

@f-peverali
Copy link
Contributor Author

  • Auf der Seite zu ISiKNachricht taucht in den Spiegelstrichen oben auf der Seite 2x "soll" in Kleinschrift auf. Das MUSS eigentlich auch groß geschrieben werden ("SOLL").

Hier gelöst 1447c73

Copy link
Contributor

@jcaumann jcaumann left a comment

Choose a reason for hiding this comment

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

passt

@f-peverali
Copy link
Contributor Author

@pavlodyban, ich würde mich zu diesem Punkt auch nochmal über Ihre Meinung freuen.


Beispiele:

```GET [base]/Appointment?encounter=Encounter/123```

Choose a reason for hiding this comment

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

Ich frage mich, ob hier die Suche richtig formuliert ist. Müsste an dieser Stelle nicht ein Reverse Search stattfinden: "Gib mir alle Fälle, die mit meinem Termin verknüpft sind"? Es gibt ja keine direkte Verbindung vom Termin auf den Fall, nur umgekehrt.

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Ja, hatte ich oben auch so geschrieben

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Danke für den Hinweis. Muss die "Reverse Suche" durch Verweis auf entsprechende Such-Funktionalität festgelegt oder anderweitig spezifiziert werden oder reicht GET [base]/Encounter?appointment=Appointment/123 mit gegebenem Verweis auf Abschnitt zu "Reference Search", @alexzautke ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Letztes reicht, da im Basismodul festgelegt ist, dass Reverse Changing über alle Referenzen hinweg funktionieren soll. Ein entsprechender Titus-Test sollte nicht vergessen werden ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alexzautke könntest du hier nochmal ausführen, was für ein test nötig wäre? (siehe Test ISIK-236)

Copy link
Contributor

Choose a reason for hiding this comment

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

@f-peverali GET [base]/Encounter?_has:Appointment:appointment:_id=Appointment/test wäre ein Beispiel für eine solche Anfrage.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@alexzautke : erzeugt hier nochmal ein Ticket für Testing. TODO nur dafür. dann kann das ticket geschlossen werden.

@f-peverali f-peverali merged commit 99d2cfe into 3.0.0-rc2 May 17, 2023
@f-peverali f-peverali deleted the enhancement/IG-chapter-Encounter branch May 17, 2023 14:07
@f-peverali f-peverali assigned alexzautke and unassigned f-peverali Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
4 participants