Skip to content

Enforce strict limits when forming the probable bounds message#115

Merged
woodwoerk merged 1 commit into
mainfrom
fix/cap-soft-limits-to-hard
Jun 12, 2026
Merged

Enforce strict limits when forming the probable bounds message#115
woodwoerk merged 1 commit into
mainfrom
fix/cap-soft-limits-to-hard

Conversation

@woodwoerk

Copy link
Copy Markdown
Collaborator

Description

Previously, if a measure had a strict limit (e.g. minValue) and a suggested limit (e.g. probableLowerBound), and the suggested limit was higher/ lower than the strict limit, it was incorrectly used in the message. Fix this so that the strict bounds are considered when forming the message.


✅ Pre-Merge Checklist

Type of Change

  • Set the PR's label to match the nature of this change

Testing

  • Built Unit tests (unit tests added/updated)
  • [-] Authorization is tested (permissions and access controls verified)
  • [-] Mobile screen widths tested for responsiveness (if applicable)
  • Manually tested locally in all affected projects (functionality verified)
    Manual testing instructions
    If feature requires manual testing by reviewer, you can provide instructions here.

Internationalization & Accessibility

  • [-] Accessibility standards met (WCAG compliance, screen reader support)

Dependencies

  • [-] Dependencies are merged (if applicable. If the change depends on other PRs)

@woodwoerk woodwoerk requested review from lilia1891 and terotik June 9, 2026 07:13
@woodwoerk woodwoerk added the Bug fix Bug fix (non-breaking change which fixes an issue) label Jun 9, 2026

@lilia1891 lilia1891 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

molto buono!

@woodwoerk woodwoerk merged commit 21f5da8 into main Jun 12, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug fix Bug fix (non-breaking change which fixes an issue)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants