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

Dependency ISiK Basis aktualisiert, Links gefixed #108

Merged
merged 3 commits into from
Jan 3, 2023
Merged

Conversation

simoneOnFhir
Copy link
Contributor

Dependency ISiK Basis aktualisiert, Links gefixed, Canonical von IG gefixed

@f-peverali
Copy link
Contributor

@simoneOnFhir Die Canonical URL ist scheinbar inkohärent zu dem was wir bisher haben, z.B.:
"https://gematik.de/fhir/ISiK/v2/ImplementationGuide/ISiK-basismodul"
oder
"https://gematik.de/fhir/ISiK/v2/ImplementationGuide/ISiK-Medikation"

waren die bisherigen canonical URLs auch falsch? Oder woher kommt die Änderung?

@simoneOnFhir
Copy link
Contributor Author

@simoneOnFhir Die Canonical URL ist scheinbar inkohärent zu dem was wir bisher haben, z.B.: "https://gematik.de/fhir/ISiK/v2/ImplementationGuide/ISiK-basismodul" oder "https://gematik.de/fhir/ISiK/v2/ImplementationGuide/ISiK-Medikation"

waren die bisherigen canonical URLs auch falsch? Oder woher kommt die Änderung?

Mir ist das aufgefallen, weil Simplifier den Canonical Mismatch auf der Projektstartseite anzeigt:
image

Mir fällt kein Grund ein, warum wir bei den V2 Modulen in den IGs absichtlich eine abweichende Canonical verwendet haben sollten. Ich versuche mal im Team zu recherchieren, ob es dafür einen Grund gab, oder ob das ein Tooling Problem ist, das bisher bloß niemandem aufgefallen ist...

@simoneOnFhir
Copy link
Contributor Author

Sieht tatsächlich so aus, als wäre das ein Modulübergreifendes Versehen gewesen, das bisher noch niemandem aufgefallen ist. Ursache vermutlich: Die Canonical der IG Ressource ist die einzige, die nicht von SUSHI automatisch gesetzt und über die Sushi-config.yaml gesteuert werden kann. Wir haben wohl irgendwann entschieden, die Base-Canoncials der Module zu ändern und es versäumt, die Canonicals der IG-Ressourcen manuell anzupassen. Sollte in den anderen Modulen in einer der nächsten TCs oder spätestens in V3 angeglichen werden.

@simoneOnFhir
Copy link
Contributor Author

tl;dr: im Dokumentenaustausch V3 ist es jetzt richtig, in den V2-Modulen ist es falsch.

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