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

Release v6.1.0 alpha.1 #440

Merged
merged 21 commits into from
Jan 24, 2025
Merged

Release v6.1.0 alpha.1 #440

merged 21 commits into from
Jan 24, 2025

Conversation

michael-82
Copy link
Collaborator

Added

  • Added translations to ui profiles (#430)

Changed

  • Change display and translation structure for criteria and concepts (#382)
  • Change codeable-concept/entry endpoint to take a list of ids as query param instead of just one id as path param (#433)
  • Replace @MockBean annotation with @MockitoBean due to deprecation (#434)
  • Replace deprecated calls to bucket4j library (#424)
  • Read ids from downloaded files in integration tests to remove the need to change those each time a new ontology is used (#428)

Removed

  • @DaTa Annotation removed from JPA classes (#332)

Fixed

  • Missing Parameters for DSE profile data added in open api doc (#431)

Security

  • Update Spring Boot to 3.4.2 (#437)

- add missing parameter in log message
…fix-code-scanning-alert-unused-format-argument

#425 - Fix code scanning alert - Unused format argument
- replace calls to Bandwith constructor with BandwithBuilder
- replace oauth token request constructor call with call to builder
…cepts

- remove nested declaration from elastic search documents (seems to be unnecessary, but should be monitored)
- modify swagger file to match new responses
- adapt tests to the new structure
- Change responses to codeable concept and terminology replies to the new structure
- change de-De to de and en-US to us for the elastic search related entries (the ones sent to the frontend are unchanged...might have to change as well)
- modify elastic search tests in integration tests
- remove GET of documents with fixed ids from elastic search integration test since those ids differ with each new ontology
- add translations to CriteriaProfileData
- adapt swagger file
…-change-display-and-translations-structure-for-criteria-and-concepts

#382 - Change display and translations structure for criteria and concepts
…tests

- use grep/awk to extract the first id from an ontology and a codeable_concept file to use for checks
…-read-document-ids-from-downloaded-files-in-github-integration-tests

#428 - Read document ids from downloaded files in github integration tests
- change hashcode method in Context class
- Remove @DaTa and @EqualsAndHashCode annotations and replace them with @Getter @Setter @tostring (excluding lazily loaded variables) and custom equals and hashcode methods
- Change id datatype from int to Long in entity classes
…-remove-data-annotation-from-jpa-classes

#332 - Remove @DaTa annotation from jpa classes
…parameters-missing-for-dse-profile-data-in-swagger-documentation

#431 - Parameters missing for dse profile data in swagger documentation
- replace @MockBean with @MockitoBean
…-replace-deprecated-mockbeans

#434 - Replace deprecated MockBeans
- update spring boot dependency
…-update-to-spring-boot-342

#437 - Update to spring boot 3.4.2
…iple ids

- Endpoint is changed to /codeable-concept/entry, expecting a list of ids as query parameter
- CodeableConceptEntry now included the id in the returned json
- Add 1 sec sleep after uploading testdata because there seems to be a timing issue
…-change-endpoint-codeable-conceptentryid-to-allow-for-multiple-ids

#433 - Change endpoint /codeable-concept/entry/{id} to allow for multiple ids
- update ontology tag to 3.0.2-alpha.2
- modify AttributeDefinition to include displayentry
…-adjust-ui-profiles-to-adjust-for-translations

#430 - Adjust ui profiles to adjust for translations
@michael-82 michael-82 added the Release For actual releases or release preparation label Jan 24, 2025
@michael-82 michael-82 self-assigned this Jan 24, 2025
Copy link

codecov bot commented Jan 24, 2025

Codecov Report

Attention: Patch coverage is 36.92308% with 123 lines in your changes missing coverage. Please review.

Project coverage is 75.26%. Comparing base (171e515) to head (bd031c0).
Report is 27 commits behind head on master.

Files with missing lines Patch % Lines
...bility_gui_backend/dse/persistence/DseProfile.java 0.00% 8 Missing ⚠️
...asibility_gui_backend/query/persistence/Query.java 0.00% 8 Missing ⚠️
...ty_gui_backend/query/persistence/QueryContent.java 0.00% 8 Missing ⚠️
...y_gui_backend/query/persistence/QueryDispatch.java 0.00% 8 Missing ⚠️
...y_gui_backend/query/persistence/QueryTemplate.java 0.00% 8 Missing ⚠️
...lity_gui_backend/query/persistence/SavedQuery.java 0.00% 8 Missing ⚠️
...y_gui_backend/query/persistence/UserBlacklist.java 0.00% 8 Missing ⚠️
...erminology/persistence/ContextualizedTermCode.java 0.00% 8 Missing ⚠️
...rsistence/ContextualizedTermCodeToCriteriaSet.java 0.00% 8 Missing ⚠️
...i_backend/terminology/persistence/CriteriaSet.java 0.00% 8 Missing ⚠️
... and 7 more
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #440      +/-   ##
============================================
- Coverage     78.54%   75.26%   -3.29%     
- Complexity      558      579      +21     
============================================
  Files           138      154      +16     
  Lines          2419     2595     +176     
  Branches        149      228      +79     
============================================
+ Hits           1900     1953      +53     
- Misses          461      576     +115     
- Partials         58       66       +8     

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

@michael-82 michael-82 merged commit 7fccff7 into master Jan 24, 2025
6 of 8 checks passed
@michael-82 michael-82 deleted the release/v6.1.0-alpha.1 branch January 24, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release For actual releases or release preparation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant