Skip to content

Conversation

@cafesilencio
Copy link

The recent addition of contours_meters for the Isochrone API resulted in a malformed URL. This is a fix for that regression.

@cafesilencio cafesilencio force-pushed the sb-NAVAND-3884-fix-isochrone-url branch from 22ee74c to da8167a Compare October 15, 2024 01:35
@cafesilencio cafesilencio marked this pull request as ready for review October 15, 2024 01:35
@cafesilencio cafesilencio requested a review from a team as a code owner October 15, 2024 01:35
@codecov
Copy link

codecov bot commented Oct 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.03%. Comparing base (f519b6f) to head (da8167a).
Report is 1 commits behind head on main.

Current head da8167a differs from pull request most recent head 2b966d2

Please upload reports for the commit 2b966d2 to get more accurate results.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1599      +/-   ##
============================================
- Coverage     78.07%   78.03%   -0.05%     
  Complexity     1066     1066              
============================================
  Files           158      158              
  Lines          4493     4493              
  Branches        630      630              
============================================
- Hits           3508     3506       -2     
  Misses          714      714              
- Partials        271      273       +2     
Files with missing lines Coverage Δ
...java/com/mapbox/api/isochrone/MapboxIsochrone.java 86.31% <100.00%> (-2.11%) ⬇️

@DzmitryFomchyn DzmitryFomchyn enabled auto-merge (squash) October 15, 2024 12:19
@DzmitryFomchyn DzmitryFomchyn merged commit cc22be7 into main Oct 15, 2024
@DzmitryFomchyn DzmitryFomchyn deleted the sb-NAVAND-3884-fix-isochrone-url branch October 15, 2024 12:23
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.

3 participants