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

correct URL of GEM_ERP_VS_DocumentType to `https://gematik.de/fhir/… #23

Closed
wants to merge 1 commit into from

Conversation

cybernop
Copy link
Contributor

Hier ein Vorschlag zur Diskussion.

Bei der Einführung der Rulesets ist aufgefallen, dass es eine Unstimmigkeit bei der URL von GEM_ERP_VS_DocumentType gibt.

Nachfolgend sind die Auswirkungen zu sehen, die eine Korrektur der URL auf https://gematik.de/fhir/erp/ValueSet/GEM_ERP_VS_DocumentType hätte.

Änderungen gäbe es in der SDs von Composition, Task und DocumentType. Auf die Beispiel-Instanzen und Echtwelt-Instanzen sollte dies meiner Meinung keine Auswirkungen haben, da sich hier auf das CodeSystem bezogen wird.

@SvenSommer
Copy link
Contributor

Angesichts der Tatsache, dass der Fehler keine direkten Auswirkungen hat, sehe ich keinen wesentlichen Vorteil in seiner Behebung. Daher empfehle ich, ihn als "bekannten Bug" in der API zu vermerken. Es ist zwar bedauerlich, dass dieser Fehler besteht, aber es birgt ein höheres Risiko und könnte theoretisch das Produktivsystem beeinträchtigen, sollten wir versuchen, diesen speziellen Fehler zu beseitigen.

Branch kann geschlossen werden als "Won't fix" und das Problem kann gerne als neues Issue in dem Repo weiterleben.

@cybernop
Copy link
Contributor Author

Wird nicht angepasst, da eine Anpassung ein deutlich höheres Risiko für das Produktivsystem darstellen kann.

@cybernop cybernop closed this Feb 12, 2024
@cybernop cybernop deleted the fix/document-type-vs-url branch February 12, 2024 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants