search

IHE / ITI.MHD

The Mobile access to Health Documents (MHD) Profile defines one standardized interface to health documents for use by mobile devices so that deployment of mobile applications is more consistent and reusable. The transactions defined here leverage the document content- and format-.agnostic metadata
https://profiles.ihe.net/ITI/MHD/index.html
Creative Commons Attribution 4.0 International
9 stars 6 forks source link

prototype fix for date range issue #141

Closed JohnMoehrke closed 1 year ago

JohnMoehrke commented 2 years ago

close #129

JohnMoehrke commented 2 years ago

This pull-request is presented as a prototype for the fix recommended by @qligier in #129

Given this issue is also recorded as CP-ITI-1254, we must either decide [ ] This is a technical correction, needing only the github issue. Thus cancel the CP [ ] This needs CP processing, so the CP processing must first happen before the change is accepted

slagesse-epic commented 2 years ago

The CP also proposes changes to XDS, so I don't think we can cancel it and should instead wait for it to be balloted.

JohnMoehrke commented 2 years ago

http://build.fhir.org/ig/IHE/ITI.MHD/branches/moehrke-date-range-fix/index.html

JohnMoehrke commented 2 years ago

The CP also proposes changes to XDS, so I don't think we can cancel it and should instead wait for it to be balloted.

Good catch. The change recommended in the CP to ITI-18 is just to add clarification text. It is not a normative change. But if that change is indeed still desirable, then yes it should survive as a CP just affecting ITI-18

JohnMoehrke commented 1 year ago

This PR is a bit more expressive than the approved ITI-CP-1254

should we downgrade the PR?

JohnMoehrke commented 1 year ago

Aligned with CP-ITI-1254-03. This was approved June 13, 2023.

Note that during alignment, I cleaned up the English as the CP was awkward wording. And I added the ge and lt clarification for emphasis.

JohnMoehrke commented 1 year ago

CI build http://build.fhir.org/ig/IHE/ITI.MHD/branches/moehrke-date-range-fix/qa.html

The warnings and info are unrelated to this pull-request. They will need to be addressed prior to release.