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

Built-in Profiles for CapabilityStatement.instantiates entry for the "supported" #1315

Closed
prb112 opened this issue Jul 1, 2020 · 1 comment
Assignees
Labels
cms-interop This issue is associated with the CMS interoperability rule profiling
Milestone

Comments

@prb112
Copy link
Contributor

prb112 commented Jul 1, 2020

Is your feature request related to a problem? Please describe.
Built-in Profiles for CapabilityStatement.instantiates entry for the "supported" implementation guides on the server.

Describe the solution you'd like

Describe alternatives you've considered
None

Additional context
None

@JohnTimm JohnTimm added the cms-interop This issue is associated with the CMS interoperability rule label Aug 25, 2020
@JohnTimm JohnTimm added this to the Sprint 18 milestone Sep 14, 2020
@lmsurpre
Copy link
Member

lmsurpre commented Oct 1, 2020

Troy and I discussed a couple different options on this one.

  1. driving the "instantiates" for a simple array of strings from the fhir-server-config.json
  2. retrieving all CapabilityStatement resources from the fhir-registry and constructing canonicals from the url+version fields of those resources

An advantage of number 1 is that it could be tenant-aware, so that each tenant can advertise a different set of IGs.
However, we already populate the profiles that are supported for each resource type from the registry (and its not tenant-aware). Also currently the validator is using all registered resources (and not just a subset).

Therefore, lets pursue option 2 at this time so that it is the most consistent with the rest of the working system.

The one place where the system is tenant-specific (wrt support for a specific IG) is the bulk data support.
However, we already have that one hardcoded in our CapabilityStatement and so we'll be no worse than we are today.
Therefore, lets update the fhir-operation-bulkdata project to provide a RegistryResourceProvider that provides just the bulkdata CapabilityStatement from https://hl7.org/fhir/uv/bulkdata/CapabilityStatement-bulk-data.html, and--if this operation jar is present--always advertise the capability (even if the tenant does not have it confiured).

We can revisit this decision if/when we implement #1449 and/or decide that we need to filter the list of all CapabilityStatements down to just some configured list (on a per-tenant basis or otherwise).

@tbieste tbieste self-assigned this Oct 1, 2020
tbieste added a commit that referenced this issue Oct 5, 2020
tbieste added a commit that referenced this issue Oct 6, 2020
tbieste added a commit that referenced this issue Oct 6, 2020
tbieste added a commit that referenced this issue Oct 7, 2020
Signed-off-by: Troy Biesterfeld <[email protected]>
prb112 added a commit that referenced this issue Oct 9, 2020
issue #1315 - instantiates field represents the registered capabilities
@tbieste tbieste closed this as completed Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cms-interop This issue is associated with the CMS interoperability rule profiling
Projects
None yet
Development

No branches or pull requests

4 participants