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

Rel 8 0 mb #6680

Merged
merged 27 commits into from
Feb 12, 2025
Merged

Rel 8 0 mb #6680

merged 27 commits into from
Feb 12, 2025

Conversation

tadgh
Copy link
Collaborator

@tadgh tadgh commented Feb 4, 2025

No description provided.

tadgh and others added 10 commits January 23, 2025 14:29
* Add test for failing use case and fix by adding Boolean property for remote validation

* changelog

* Revert not yet supported change
…6598)

* add tests and fix mass ingestion and patient compartment interceptor compatibility in mass ingestion mode

* changelogs

* address review - add parameter class for method with 10 params

* spotless

---------

Co-authored-by: jdar <[email protected]>
Co-authored-by: Kevin Dougan <[email protected]>
#6667)

* SearchParameter validation was not being skipped on updates, even if requested. This has been fixed

* spotless fixes

* removed unecessary code

* reverting the code back

* added external JIRA ref
Copy link

github-actions bot commented Feb 4, 2025

Formatting check succeeded!

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.58%. Comparing base (c096c31) to head (78537ba).
Report is 25 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6680      +/-   ##
============================================
+ Coverage     83.52%   83.58%   +0.06%     
- Complexity    28600    28650      +50     
============================================
  Files          1798     1798              
  Lines        111300   111363      +63     
  Branches      13977    13982       +5     
============================================
+ Hits          92958    93080     +122     
+ Misses        12354    12280      -74     
- Partials       5988     6003      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

mrdnctrk and others added 15 commits February 4, 2025 22:35
…6684)

* made replace-references not replace versioned references
…cher (#6691)

* failing test

* narrow failing test

* fix

* changelog

* clean up

* spotless
…d as a valid sort spec with Elastic Search enabled. (#6670)

* added test for '-delt-add' in hapi

* fixed an issue with the _lastUpdated search param not being recognized as a valid sort spec with Hsearch enabled in CDR

* added null check

* added tests

* added comments

* addressed comments

* updated static class property name

* modified test

* added changelog and addressed more comments

* ran mvn spotless

* addressed comments

* ran mvn spotless

* addressed comments

---------

Co-authored-by: Abayomi Adebowale <[email protected]>
…fault partition id is not null (#6687)

* initial test, solution and changelog

* addressing comments from code review.

* addressing code review comments and fixing tests.

* addressing code analysis issues.

* addressing code analysis issues.

* fixing failing tests

---------

Co-authored-by: peartree <[email protected]>
* Account for case differences in DBPM contributor

* Test fix

* Cleanup
* Add flag to disable schema check

* Spotless
* Make the Canonicalizer non-null-default-partition-id aware

* Rename test

* Changelog

* Rework

* Remove dead IT

* remove fixme
@tadgh tadgh merged commit 066c242 into master Feb 12, 2025
77 of 121 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.