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.

Sign up for GitHub

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

Jump to bottom

Inbox: Phase aus der Vergangenheit angewählt #1228

Closed
St-Annette opened this issue Feb 14, 2022 · 28 comments
Closed

Inbox: Phase aus der Vergangenheit angewählt #1228

St-Annette opened this issue Feb 14, 2022 · 28 comments
Assignees
@maennchen
Labels
bug Something isn't working
Milestone
v1.45.0

Comments

@St-Annette
Copy link

St-Annette commented Feb 14, 2022 •
edited by maennchen
Loading

Describe the bug

Inbox: Beim Import wird bei einer Person ein Fall ausgewählt der im Dezember abgeschlossen war.
Der Abgleich wurde als grün marktiert.

To Reproduce

  1. Import
  2. Zeile abgleichen
  3. Abzugleichender Fall erscheint in grün
  4. Der Fall ist nicht aktuell
  5. Isolation wird unterdrückt (nicht korrekt)
  6. Autotracing wird nicht ausgelöst (nicht korrekt)
  7. Es würde kein neuer Fall erstellt werden (nicht korrekt)

Expected behavior

  1. Person wird ausgewählt (grün)
  2. Fall wird nicht ausgwählt
  3. AT wird verschickt
  4. Neuer Fall wird erstellt

Screenshots

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

References

Link zum Fall:
https://covid19-tracing.ch/cases/5f133be3-1c72-42d1-93d9-3c70341133a9

(Screenshot wird per Mail geschickt)
@St-Annette St-Annette added the bug Something isn't working label Feb 14, 2022
@maennchen
Copy link
Contributor

@St-Annette Kannst du mir noch den Link zur Import-Zeile schicken?

@maennchen
Copy link
Contributor

@rebecca-br Schaut ob das noch öfter auftritt und schickt mir Zeilen ID und Fall ID zm Fehler. Die Zeile darf weiter importiert werden, ich kann das auch danach nachvollziehen.

@rebecca-br
Copy link

@maennchen bei diesem Fall wurde beim Import ein existierender Fall angezeigt mit einer Phase vom November. Die Phase vom November habe ich in der Inbox dann gelöscht, damit ein ein neuer Fall erstellt wird. Hier den Link dazu:

https://covid19-tracing.ch/cases/41c7c055-fb9f-486b-b944-2427f32b4adc

@rebecca-br
Copy link

@maennchen was ist die Zeilen ID?

@maennchen
Copy link
Contributor

@rebecca-br Die ID ist Teil der URL. Am besten schickst du mir einfach die URL der Zeile.

@rebecca-br
Copy link

@maennchen Hier einige Fälle wo bei der Inbox ein existierender Fall vom November oder Dezember angezeigt wird.

image
image
image
image

@maennchen
Copy link
Contributor

@rebecca-br Kannst du die URL's als Text senden und nicht als Bild?

@St-Annette
Copy link
Author

St-Annette commented Feb 19, 2022 •
edited
Loading

@maennchen Hier wird heute, 19.02., der Fall ausgewählt, der am 13.02. endet. Der Abgleich-Fall steht auf grün.
https://covid19-tracing.ch/inbox/rows/ba79fc8e-5dd1-4dac-bd43-ed9b808ee5bd/apply
https://covid19-tracing.ch/people/508aae67-0527-41f7-8352-eec239d93cea

Hier wird heute, 19.02., ein Fall ausgewählt, der am 02.02.22 endet. Ebenfalls auf grün
https://covid19-tracing.ch/inbox/rows/ffc90e66-73cf-4bba-a2a3-dd5ec0a5272f/apply
https://covid19-tracing.ch/people/2ad78ee4-4f35-456b-93c2-ad30ca9d8202

Hier wird ebenfalls heute ein Fall ausgewählt, der am 03.12.21 endet. Ebenfalls auf grün
https://covid19-tracing.ch/inbox/rows/92bea3a4-f0d6-4552-8417-7174b7165585/apply
https://covid19-tracing.ch/people/c812d206-f7ea-4da2-aa0f-9b4298dca7fd

Hier wird heute ein Fall ausgewählt, der am 16.02. endet. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/2cb040d7-9779-4c9a-a8ae-7a9deed587cc/apply
https://covid19-tracing.ch/people/7b985cad-3fb7-4f33-be49-2d67a9b1f142

Hier wird heute ein Fall ausgewählt, der am 01.12.21 endet. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/c885b7dd-339f-48b5-9b86-a655cbd04630/apply
https://covid19-tracing.ch/people/3a171179-e184-405f-9640-d8baf9abdefd

Hier wird heute ein Fall ausgewählt, der am 01.12.21 endet. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/3112e970-82bc-45db-a65c-17418827537a/apply
https://covid19-tracing.ch/people/5d666ada-c178-4dc3-97ae-f7b44b2300c2

Hier wird heute ein Fall ausgewählt, der am 23.11.21 erstellt worden ist und auf abgebrochen steht. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/dcd0066c-f6f7-4cdb-9384-c31efa2d16fa/apply
https://covid19-tracing.ch/cases/1b233115-b04b-4b18-a30b-a5f62786e35e

Hier wird heute ein Fall ausgewählt, der am 12.02.22 endet. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/6818b29e-241f-48e6-8748-5549bf9ba5dc/apply
https://covid19-tracing.ch/people/9a78e22e-2001-450d-86c4-f427c8b17730

Hier wird heute ein Fall ausgewählt, der am 04.12.21 endet. Ebenfalls grün.
https://covid19-tracing.ch/inbox/rows/f209bdd0-ef2c-40c9-bed2-e93fe65c14ad/apply
https://covid19-tracing.ch/people/b4f852e1-eafc-46a3-a63e-7c61e63546fa

@rebecca-br
Copy link

@maennchen sorry jetzt sollte es klappen.

https://covid19-tracing.ch/inbox/rows/99bd8a21-e1ac-4a77-83ac-842a82b3102b

https://covid19-tracing.ch/inbox/rows/5c4defb0-dba6-4013-9d3f-90b5105051fd

https://covid19-tracing.ch/inbox/rows/3faafe68-e75f-4164-8540-c749c2378ffa

https://covid19-tracing.ch/inbox/rows/b6f427dd-a1cf-4979-bf2e-9116990be579

https://covid19-tracing.ch/inbox/rows/3e5fa9e8-04d6-4966-a5c4-e1f0809fe8fc

@St-Annette St-Annette assigned maennchen and unassigned St-Annette and rebecca-br Feb 19, 2022
@janmrnak
Copy link
Contributor

Hallo zusammen, ich konnte bisschen reinschauen und bin überzeugt, dass das Problem entsteht wenn die Fall ID Nummer die gleiche ist wie beim alten Fall.

@maennchen Ich kann es lokal nachvollziehen, alter Fall, gleiche Fall ID und Testdatum im Format "2022-02-16" (nicht "16.02.2022"!!) in der test.csv

@maennchen
Copy link
Contributor

@janmrnak Die haben die gleiche Fall-Id? Das wäre dann sicher ein Problem im ISM.

@St-Annette
Copy link
Author

@maennchen Ja das habe ich gerade auch gesehen.
Auch wenn es letztlich ein Fehler vom ISM ist, fehlen uns nachher die Fälle im System.
Wir sollten also trotz gleicher Fall ID validieren:

Phase bereits abgeschlossen (Ende liegt in der Vergangenheit):

  • Fall wird ausgewählt
  • Abgleich wird rot angezeigt
  • Isolation / Quarantäne wird unterdrückt

Evtl. könnte man das Ticket mit #488 umsetzen. @rebecca-br ?

@janmrnak
Copy link
Contributor

https://app.forecast.it/project/P-205/scoping/T3623

Offen:
Fall wird ausgewählt egal wie lang das Ende in der Vergangenheit liegt?

Umsetzung mit #488 wäre sinnvoll.

@rebecca-br
Copy link

@maennchen @St-Annette @janmrnak ja dann setzen wir das mit dem um mit dem Ticket #488 zusammen.

@maennchen
Copy link
Contributor

@rebecca-br Das Issue #488 hängt nicht mit diesem zuammen. Wir können es aber auch gerne noch einschätzen.

@rebecca-br
Copy link

@maennchen dann bitte einfach dieses Ticket umsetzen. Danke

@janmrnak janmrnak mentioned this issue Feb 23, 2022
Merged
@maennchen maennchen added this to the v1.45.0 milestone Feb 24, 2022
@maennchen
Copy link
Contributor

@rebecca-br / @saanni / @St-Annette Bereit zum testen.

@maennchen maennchen assigned janmrnak and unassigned saanni Feb 25, 2022
@rebecca-br
Copy link

@maennchen alles klar.

maennchen added a commit that referenced this issue Feb 28, 2022
@maennchen
Inbox Check Dates for Rows with Predecessor (#1228)
658c837
@maennchen maennchen mentioned this issue Feb 28, 2022
Merged
@maennchen maennchen assigned maennchen and unassigned janmrnak Feb 28, 2022
maennchen added a commit that referenced this issue Feb 28, 2022
@maennchen
Inbox Check Dates for Rows with Predecessor (#1228) (#1275)
4d09378
@maennchen
Copy link
Contributor

@rebecca-br / @saanni / @St-Annette Bereit für einen erneuten Test.

@rebecca-br
Copy link

@maennchen

Phase bereits abgeschlossen (Ende liegt in der Vergangenheit):

Fall wird ausgewählt
Abgleich wird jetzt rot angezeigt.

Dann würde das jetzt stimmen.

@maennchen
Copy link
Contributor

@rebecca-br Soll ich den Release machen über Mittag?

@rebecca-br
Copy link

@maennchen ja gerne.

@maennchen
Copy link
Contributor

@rebecca-br Release ist live.

@rebecca-br
Copy link

@maennchen perfekt Vielen Dank.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maennchen maennchen

Labels
bug Something isn't working
Projects
None yet
Milestone
v1.45.0
Development

No branches or pull requests
5 participants
@maennchen @janmrnak @St-Annette @rebecca-br @saanni