Skip to content

Conversation

@optionsome
Copy link
Member

Summary

This is a small refactoring to simplify the internal model of visit via locations to explicitly only allow one coordinate as the APIs don't support multiple coordinates.

Issue

Related to #4887

Unit tests

Updated tests.

Documentation

Not updated

Changelog

Skipped

@optionsome optionsome added this to the 2.9 (next release) milestone Dec 19, 2025
@optionsome optionsome requested a review from a team as a code owner December 19, 2025 21:02
@optionsome optionsome added !Technical Debt Improve code quality, no functional changes. +Skip Changelog This is not a relevant change for a product owner since last release. labels Dec 19, 2025
@codecov
Copy link

codecov bot commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.12%. Comparing base (9aa668b) to head (3809146).
⚠️ Report is 1 commits behind head on dev-2.x.

Files with missing lines Patch % Lines
...uting/algorithm/mapping/ViaLocationTypeMapper.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             dev-2.x    #7152   +/-   ##
==========================================
  Coverage      72.12%   72.12%           
- Complexity     20837    20838    +1     
==========================================
  Files           2268     2268           
  Lines          84338    84335    -3     
  Branches        8415     8415           
==========================================
- Hits           60827    60826    -1     
+ Misses         20538    20536    -2     
  Partials        2973     2973           

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

+Skip Changelog This is not a relevant change for a product owner since last release. !Technical Debt Improve code quality, no functional changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant