Skip to content

Internt review af IG#5

Open
tmsMedcom wants to merge 6 commits intomainfrom
review-internal
Open

Internt review af IG#5
tmsMedcom wants to merge 6 commits intomainfrom
review-internal

Conversation

@tmsMedcom
Copy link
Contributor

Review af IG

@tmsMedcom tmsMedcom requested a review from SGA-MedCom February 9, 2026 10:54
@github-actions
Copy link

github-actions bot commented Feb 9, 2026

QA Report status 🚨

🔨 https://build.fhir.org/ig/medcomdk/dk-medcom-notes/branches/review-internal 🔨
Errors: 12, Warnings: 39, Info: 59, Broken links: 0

@RikkeVestesen
Copy link

RikkeVestesen commented Feb 26, 2026

Hej med jer.

Her er noter fra Søren og min ift. gennemgang af IGen :)

Hvorfor er det mimeType xhtml+xml? Vi foreslår text/html

Det er profileret sådan, at hvis man har en author, så skal det være en practitionerRole med reference til en practitioner. Kan man mon have en practitioner som author uden?

Composition.date: Denne har ingen invariant på omkring formatet - skal man kunne sende uden tidspunkt, eller skal der en invariant på? Og hvilken tidszone må man bruge?
Er medtaget i Observation.effective

Composition.event.period.start/end: Disse bør have samme invariant på som EKG har, da det mapper over i XDS-metadata. Der mangler måske også lidt mere beskrivelse på hvad det for et tidspunkt man skal indsætte i disse felter.

I EKG kan man både have en author i composition som Practitioner og PractitionerRole. Er det med vilje at I har begrænset til en Practitioner?

Jeg kan se I har linket til specifikke versioner af profiler, når I linker til de profiler I bruger fra andre IGer. Det giver principielt rigtig god mening, men det skal skabe fejl ved opdateringer af standarden. I EKG har vi skrevet en note om det i stedet v. links væk fra IGen. Jeg ved ikke hvad der er den gode løsning, blot et OBS punkt :)

Vær OBS på fejl i QA-rapporten - jeg tolker det som at f.eks. typecode ikke er inkluderet korrekt endnu, men der er lidt forskelligt at se på i rapporten.

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.

3 participants