5089 cr measure refactor #5324
Merged
5089 cr measure refactor #5324
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Bump to core release 6.0.16 * Bump to core version 6.0.20 * Fix errors thrown as a result of VersionSpecificWorkerContextWrapper * Bump to core 6.0.22
…automatically indexes the primary key (#5127) * dropped old index FK_RESVERPROV_RES_PID on RES_PID column before adding IDX_RESVERPROV_RES_PID * added changelog * changed to valid version number * changed to valid version number, need to be ordered by version number...
…ied (#5124) 5123 - Use DEFAULT partition for server-based requests if none specified
* added FIXME with source of issue * added FIXME with root cause * added FIXME with root cause * Providing solution to the issue and removing fixmes. * Providing changelog * auto-formatting. * Adding new test. * Adding a new test for standard paging * let's try this and see if it works...? * fix tests * cleanup to trigger a new run * fixing tests --------- Co-authored-by: Ken Stevens <[email protected]> Co-authored-by: peartree <[email protected]>
…ore (#5118) * fix, test, changelog * fix, test, changelog --------- Co-authored-by: justindar <[email protected]>
_source search parameter needs to support modifiers - added support form :contains, :missing, :above modifiers
…r/hapi-fhir into 5089-cr-measure-refactor
|
Formatting check succeeded! |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #5324 +/- ##
============================================
+ Coverage 81.32% 82.83% +1.51%
- Complexity 23650 26303 +2653
============================================
Files 1425 1612 +187
Lines 86399 102217 +15818
Branches 11677 13067 +1390
============================================
+ Hits 70265 84676 +14411
- Misses 10947 11944 +997
- Partials 5187 5597 +410 ☔ View full report in Codecov by Sentry. |
JPercival
requested changes
Sep 20, 2023
hapi-fhir-storage-cr/src/main/java/ca/uhn/fhir/cr/common/IDaoRegistryUser.java
Show resolved
Hide resolved
hapi-fhir-storage-cr/src/main/java/ca/uhn/fhir/cr/config/TerminologyCacheConfig.java
Outdated
Show resolved
Hide resolved
.../main/java/ca/uhn/fhir/cr/dstu3/activitydefinition/ActivityDefinitionOperationsProvider.java
Show resolved
Hide resolved
JPercival
approved these changes
Sep 21, 2023
hapi-fhir-storage-cr/src/test/java/ca/uhn/fhir/cr/r4/TestCrR4Config.java
Show resolved
Hide resolved
hapi-fhir-storage-cr/src/test/java/ca/uhn/fhir/cr/r4/TestCrR4Config.java
Show resolved
Hide resolved
|
question: The test coverage for R4 is significantly better than for DSTU3. Is that just a matter of us not having content for DSTU3? |
barhodes
approved these changes
Sep 22, 2023
lukedegruchy
approved these changes
Sep 22, 2023
hapi-fhir-storage-cr/src/main/java/ca/uhn/fhir/cr/r4/measure/MeasureService.java
Show resolved
Hide resolved
hapi-fhir-storage-cr/src/main/java/ca/uhn/fhir/cr/repo/SearchConverter.java
Outdated
Show resolved
Hide resolved
hapi-fhir-test-utilities/src/main/java/ca/uhn/fhir/test/utilities/HtmlUtil.java
Outdated
Show resolved
Hide resolved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#5089