Skip to content

Conversation

@adamfilep
Copy link
Contributor

@adamfilep adamfilep requested review from apeteri and cmark October 14, 2025 16:15
@adamfilep adamfilep self-assigned this Oct 14, 2025
@codecov
Copy link

codecov bot commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.43%. Comparing base (70dbfb3) to head (536bb48).
⚠️ Report is 6 commits behind head on 9.x.

Additional details and impacted files
@@            Coverage Diff            @@
##                9.x    #1422   +/-   ##
=========================================
  Coverage     64.42%   64.43%           
- Complexity    12806    12807    +1     
=========================================
  Files          1620     1620           
  Lines         57803    57808    +5     
  Branches       5685     5685           
=========================================
+ Hits          37242    37246    +4     
- Misses        18191    18192    +1     
  Partials       2370     2370           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@apeteri apeteri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

* Description is preferred in language or dialect
*/
PREFERRED(Concepts.REFSET_DESCRIPTION_ACCEPTABILITY_PREFERRED);
PREFERRED(Concepts.REFSET_DESCRIPTION_ACCEPTABILITY_PREFERRED, "Preferred");
Copy link
Member

@cmark cmark Oct 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this is okay for en displays, it might not be okay for others in the future, so I expect some additional improvement here - in a future Snow Owl version - as well to fetch the label of the acceptability from the db instead of hardcoding it here.

Copy link
Member

@cmark cmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥓

@cmark cmark merged commit 2d2261d into 9.x Oct 15, 2025
5 checks passed
@cmark cmark added the bug label Oct 15, 2025
@cmark cmark deleted the fix/SO-6343-valueset-include-designations-display-bug branch October 15, 2025 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants