Umstellung Quarantäne auf Incidents #463
Conversation
# Conflicts: # client/src/api/SwaggerApi.ts # server/src/main/java/de/coronavirus/imis/config/SwaggerConfig.java
|
➡️ Live preview at url: https://imis-pr-463-lfq7hnbpwq-ew.a.run.app 🆙 |
… quarantine phases are now possible per patient
There was a problem hiding this comment.
Ich habe die Änderungen überflogen und in der Live-Preview getestet und das Ergebnis sieht für mich gut aus 👌. Ich kann aber wenig zur Implementierung der Incidents sagen, da ich mich mit dem Incident-System bisher kaum beschäftigt habe. Da kann @LeonardKoll sicher eher was zu äußern.
| || latest.get().getUntil().isAfter(LocalDate.now())); | ||
| } | ||
|
|
||
| public QuarantineIncident save(QuarantineIncident quarantineIncident) { |
There was a problem hiding this comment.
Ist schon atomar, daher macht die Transaction Annotation keinen Unterschied.
|
Kann RequestQuarantineDTO nicht weg? |
|
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. Was einwandfrei funktioniert ist, dass sich ein neuer Incident öffnet wenn Patienten in Quarantäne gesendet wurden und dann erneut vorgemerkt werden. |
|
@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. |
|
Ja genau! Perfekt. |
Dieser PR stellt Quarantänemaßnahme vollständig auf Incidents um.
Resolves #460