search

hl7dk / dk-core

7 stars 6 forks source link

Feature ipa support #132

Closed jkiddo closed 2 months ago

jkiddo commented 3 months ago

Original PR closed due to branch renaming. Continuing here from https://github.com/hl7dk/dk-core/pull/76

jkiddo commented 3 months ago

I would like to suggest that we do sort of like what the Finnish affilate has done (see https://hl7.fi/fhir/). Make DK Core compliant profilewise but keep IPA security aspects in its own separate IG like https://hl7.fi/fhir/finnish-smart/

@jacand how does that sound to you?

jkiddo commented 3 months ago

That is also a good question for you @ovi-medcom & @christiangasser-lakeside 👆

jacand commented 3 months ago

I would like to suggest that we do sort of like what the Finnish affilate has done (see https://hl7.fi/fhir/). Make DK Core compliant profilewise but keep IPA security aspects in its own separate IG like https://hl7.fi/fhir/finnish-smart/

@jacand how does that sound to you?

Sounds good - with the possible exception of DkCoreObservation, which we may have to split in two? IPAObservations always have Patients as subjects, while we decided to keep DkCoreObservation open to other classes of subjects. -- and there is no multiple inheritance. We need to discuss how to move this one forward...

jkiddo commented 2 months ago

we should probably follow this as well: https://github.com/HL7/fhir-ig-publisher/issues/884

jkiddo commented 2 months ago

@jacand - I've introduced a new Observation profle - but maybe it isn't needed. That is still an open thing to consider. @tmsMedcom I think we should take up this PR for next available SIG meeting (and maybe retarget it to 3.3.0 branch).

jkiddo commented 2 months ago

The IPA minimum requirements are very reasonable and we can leave out the SMART part and isolate that in its own IG ( early draft is here: https://build.fhir.org/ig/jkiddo/dk-smart/branches/master/index.html )

jkiddo commented 2 months ago

Which is comparable with the Finnish way