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

[Draft] Support for _contained=true search parameter #5792

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

Conversation

codeforgreen
Copy link
Collaborator

No description provided.

Copy link

Formatting check succeeded!

Copy link

codecov bot commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.38%. Comparing base (497b9f2) to head (bb7434c).
Report is 33 commits behind head on master.

❗ Current head bb7434c differs from pull request most recent head adc28cc. Consider uploading reports for the commit adc28cc to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5792      +/-   ##
============================================
- Coverage     83.39%   83.38%   -0.01%     
- Complexity    26927    26930       +3     
============================================
  Files          1681     1681              
  Lines        103965   104000      +35     
  Branches      13189    13193       +4     
============================================
+ Hits          86702    86725      +23     
- Misses        11613    11622       +9     
- Partials       5650     5653       +3     

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

nathandoef and others added 18 commits April 8, 2024 13:46
* failing test

* fix

* changelog

* spotless

* fix tests
…irHibernateJpaDialect (#5791)

* prefixed the log message with "Unit test: "

* resolved review comments
…ing (#5803)

* wip

* cleaning up test

* adding changelog and passing spotless.

---------

Co-authored-by: peartree <etienne.poirier@smilecdr.com>
* Reducer LOB usage in Batch2 and Search (#5748)

* Reducer LOB usage in Batch2 and Search

* Add changelog

* Rework a number of LOB columns

* Test fix

* Test fix

* Column fixes

* Test fix

* Formatting

* Fixes

* patching bad oracle test

* Apply spotless

---------

Co-authored-by: Tadgh <garygrantgraham@gmail.com>

* Clean up

* Revert change

* One more revert

* Resolve compile issue

---------

Co-authored-by: Tadgh <garygrantgraham@gmail.com>
* Add support for latest version of CR and new $questionnaire operation

* Fix formatting in javadoc causing warning

* Update to 3.0.0 CR release

* Add support for expected parameter names from updated IG's

* Update Clinical Reasoning documentation

* Update pom.xml

* Create 5750-update-cr-operations.yaml

* fix doc

* Update to latest CR version

* Update pom.xml

* In version

* Update changelog

* Remove commented code
* Limit bulk export chunk size

* Cleanup

* Work on tests

* Working

* Add some docs

* Address revuew comments

* Spotless

* Test fix
* Add _language param to providers and tests

* add pr number to docs

* fix test

* remove unnecessary code

---------

Co-authored-by: Lila Mikalson <lila.mikalson@smilecdr.com>
* update model

* convert results

* migration

* extend test coverage in cli

* fix broken test

* change log

* code review feedback

* spotless
Stop writing to hfj_forced_id
Stop reading ForcedId
Cleanup client-id assignment
Fix query counts in tests
Drop the unused indexes and fk
…ify the capability statement (#5814)

* added new customizer filter factory.  have not yet deleted old code it replaces.

* added new customizer filter factory.  have not yet deleted old code it replaces.

* replaced websocket filter.  works.  still some cleanup to do

* replaced websocket filter.  works.  still some cleanup to do

* cosmetic change

* add coverage and fix bugs it found

* spotless

* move capability statement classes

* add changelog and rename new classes

* review feedback
* step one of unifying the bulk exports

* adding changelog

* review points

* spotless

---------

Co-authored-by: leif stawnyczy <leifstawnyczy@leifs-mbp.home>
* refactoring

* test refactor only

* spotless

* bumping version

---------

Co-authored-by: leif stawnyczy <leifstawnyczy@leifs-mbp.home>
* licenses

* remove mdm placeholder interceptor

* remove mdm placeholder interceptor

* fix regression

* Remove RTE Placeholder Interceptor.
Add tests to assert proper boot sequence.

* consolidate LogbackCaptureTestExtension (removed the cdr class)

* consolidate LogbackCaptureTestExtension (removed the cdr class)
* adding a catch for failing reduction step

* spotless

* added changelog

---------

Co-authored-by: leif stawnyczy <leifstawnyczy@leifs-mbp.home>
@codeforgreen codeforgreen requested a review from a team as a code owner April 8, 2024 20:47
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.

None yet