-
Notifications
You must be signed in to change notification settings - Fork 41
Use HAPI Repository interface #654
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
Changes from all commits
1ea91dc
a3c3de7
4d6ac8a
f401f3b
57a5f43
a9f5f60
2ee5e15
2b7a430
a522ff7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,13 +3,13 @@ | |
| import static org.opencds.cqf.fhir.test.Resources.getResourcePath; | ||
|
|
||
| import ca.uhn.fhir.context.FhirContext; | ||
| import ca.uhn.fhir.repository.Repository; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What happens if we need to update this interface? Will we now have to wait at least one more release cycle to propagate it to clinical-reasoning + cdr-cr? Meaning, with a dependency sequence of hapi-fhir <- clinical-reasoning <- cdr-cr.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, and we already have that dependency for the models. Ideally this interface will not be changing often, if at all. |
||
| import java.nio.file.Paths; | ||
| import java.time.ZonedDateTime; | ||
| import java.util.function.Supplier; | ||
| import org.hl7.fhir.r4.model.Bundle; | ||
| import org.hl7.fhir.r4.model.IdType; | ||
| import org.hl7.fhir.r4.model.MeasureReport; | ||
| import org.opencds.cqf.fhir.api.Repository; | ||
| import org.opencds.cqf.fhir.cql.engine.retrieve.RetrieveSettings.SEARCH_FILTER_MODE; | ||
| import org.opencds.cqf.fhir.cql.engine.retrieve.RetrieveSettings.TERMINOLOGY_FILTER_MODE; | ||
| import org.opencds.cqf.fhir.cql.engine.terminology.TerminologySettings.VALUESET_EXPANSION_MODE; | ||
|
|
||
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did this requirement come from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've been working towards moving this interface down into HAPI for awhile now. That happened with the 8.0.0 release.