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

Coverage identifier inkonsistent #250

Closed
1 task
markus020 opened this issue Apr 13, 2023 · 3 comments · Fixed by #324
Closed
1 task

Coverage identifier inkonsistent #250

markus020 opened this issue Apr 13, 2023 · 3 comments · Fixed by #324
Assignees
Labels
AG bug Something isn't working invalid This doesn't seem right

Comments

@markus020
Copy link

markus020 commented Apr 13, 2023

Im Profil ISiKVersicherungsverhaeltnisGesetzlich sind die identifier inkonsistent:

  1. die Definition des identifier sagt, dass primär die 30-stellige Krankenversicherungsnummer verwendet werden soll und die 10-stellige KVNR nur, wenn diese unbekannt ist. Das beißt sich aber mit den Kardinalitäten -> fehlt hier ein Slice für die 30-stellige?
  2. bei beneficiary sagt der Kommentar, dass die Angabe der 10-stelligen KVNR verpflichtend ist, aber das spiegelt sich nicht in der Kard. wieder (hier wäre tatsächlich dann die 10-stellige KVNR 2x aufgeführt) -> Auflösung mittels Constraint?

Zusätzlich ist der zugehörige Testfall ebenfalls inkonsistent (der Patient hat in der Patientenressource eine andere 10-stellige KVNR als in der Coverage)

Siehe weitere Informationen in der Zulip-Diskussion: https://chat.fhir.org/#narrow/stream/287581-german.2Fisik/topic/.5BBasis.5D.20Coverage/near/349073762

Tasks

Preview Give feedback
@f-peverali f-peverali self-assigned this Apr 17, 2023
@f-peverali f-peverali added the bug Something isn't working label Apr 17, 2023
@f-peverali f-peverali moved this from Todo to InDiscussion in ISiK/ISiP Project Board Apr 17, 2023
@f-peverali
Copy link
Contributor

FYI @jcaumann

@f-peverali f-peverali removed their assignment Sep 18, 2023
@f-peverali f-peverali moved this from InDiscussion to Todo in ISiK/ISiP Project Board Sep 18, 2023
@f-peverali f-peverali added the invalid This doesn't seem right label Sep 18, 2023
@f-peverali f-peverali moved this from Todo to Backlog in ISiK/ISiP Project Board Sep 19, 2023
@f-peverali f-peverali moved this from Backlog to Todo in ISiK/ISiP Project Board Oct 16, 2023
@f-peverali
Copy link
Contributor

f-peverali commented Oct 26, 2023

@patrick-werner, @alexzautke
zu Punkt 1. würde ich diesem ticket (https://hl7germany.atlassian.net/browse/HFB-57) folgen und den entsprechenden Text auch in unserem Profil/IG entfernen ? Also "Ein gesetzliches Versicherungsverhältnis sollte stets durch die eindeutige 30-stellige Versichertennummer identifiziert werden... "etc

Zu 2.: Verstehe ich es richtig, dass hier eine eine informative Änderung hinzu sollte im Sinne dieses Tickets? https://hl7germany.atlassian.net/browse/HFB-58

Grundsätzlich scheint der Bedarf nach einer robusteren ID gegenüber der 10-Stelligen noch nicht aufgelöst, wenn ich mir das hier ansehe: https://chat.fhir.org/#narrow/stream/287581-german.2Fisik/topic/.5BBasis.5D.20Coverage/near/349073762
Sollten wir das nochmal Aufgreifen in der Bearbeitung?

Synchron zu 1 entschieden: ja, "30 ... soll raus"

@f-peverali f-peverali added the AG label Oct 26, 2023
@f-peverali f-peverali self-assigned this Oct 26, 2023
@patrick-werner
Copy link
Contributor

patrick-werner commented Oct 26, 2023

Nach heutiger Diskussion wurden folgende technischen Todos beschlossen:

  • subscriber & subscriber.identifier werden MS
  • beneficiary ^comment ist falsch: die Angabe der reference sollte verpflichtend sein, der Identifier ist optional (auch kein MS)

@f-peverali f-peverali moved this from Todo to In Progress in ISiK/ISiP Project Board Nov 8, 2023
@f-peverali f-peverali linked a pull request Nov 10, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from In Progress to Done in ISiK/ISiP Project Board Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AG bug Something isn't working invalid This doesn't seem right
Projects
Development

Successfully merging a pull request may close this issue.

3 participants