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

Provide attributes for storing values relevant for XDS Metadata Update #234

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

unixoid
Copy link

@unixoid unixoid commented Jul 1, 2024

Closes #

📑 Description

This PR introduces changes which enable a mapping between MHD DocumentReference resources and XDS Metadata Update requests. The following XDS attributes are in focus:

  • logicalID
  • repositoryUniqueID
  • version
  • documentAvailability

The first one is mapped to a dedicated slice of DocumentReference.identifier, the latter three -- to extensions.

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • I have selected a committee co-chair to review the PR

ℹ Additional Information

@JohnMoehrke
Copy link
Contributor

Committee discussion:

There needs to be an Issue created that explains why these changes are needed. Please explain the Use-Case, and why this can't be solved without a change to IHE-MHD.

Would support for these elements be needed by all MHD actors? Or would there be an named Option defined for support of these elements? Given that the majority of MHD actors will never need these elements, it seems not appropriate to require support for these elements on all Actors.

When is it allowed to have these elements populated? Are they allowed to be populated in ITI-65? I would say not.

The scale of this change will need to be done with a CP. Having the Issue and PR available is a benefit to the CP process. We need the CP process for the sake of governance and visibility.

@JohnMoehrke JohnMoehrke added the question Further information is requested from issue author label Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested from issue author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants