Umstellung Quarantäne auf Incidents

ImisDevelopers / 1_011_a_infektionsfall_uebermittellung

This repository is deprecated and has not been maintained for months!

https://imis-prototyp.de/

MIT License

22 stars 6 forks source link

Umstellung Quarantäne auf Incidents #463

Open btobi opened 4 years ago

btobi commented 4 years ago

Dieser PR stellt Quarantänemaßnahme vollständig auf Incidents um.

Das Frontend kommuniziert ausschließlich über Incidents mit dem Backend
Redundante Informationen aus dem Patienten wurden entfernt
Ein Patient kann nun mehrfach in Quarantäne geschickt werden.

Resolves #460

github-actions[bot] commented 4 years ago

:arrow_right: Live preview at url: https://imis-pr-463-lfq7hnbpwq-ew.a.run.app :up: :warning: First access might be very slow, database is shared. :eyes:

LeonardKoll commented 4 years ago

Kann RequestQuarantineDTO nicht weg?

LeonardKoll commented 4 years ago

Wenn ich für den selben Patienten zwei mal unmittelbar hintereinander Quarantäne vormerke (mit dem selben "bis" Datum, dann entstehen im Backend zwei Incidents. Beim "In Quarantäne senden" kommt es dann infolgedessen zu einem Fehler. Erwartetes Verhalten beim zweiten Vormerken: Das Backend prüft, ob für den Patienten eine offene Anordnung / Vormerkung besteht. Diese wird angepasst (aktualisiert).

Was einwandfrei funktioniert ist, dass sich ein neuer Incident öffnet wenn Patienten in Quarantäne gesendet wurden und dann erneut vorgemerkt werden.

btobi commented 4 years ago

@LeonardKoll Gut dass du das gefunden hast. Habe das jetzt gelöst, indem ich mir den letzten Quarantäne Incident hole und prüfe, ob das ein SELECTED oder MANDATED ist. Tendenziell sollte das Frontend den QuarantäneIncident selbst schon vorliegen haben, aber ich denke das ist dann auch eine Frage vom Umarbeiten der UI.

LeonardKoll commented 4 years ago

Ja genau! Perfekt.