Skip to content

PB-1007: fix doubling of length value#1069

Merged
ltkum merged 1 commit intodevelopfrom
fix-PB-859-doubling-length
Sep 17, 2024
Merged

PB-1007: fix doubling of length value#1069
ltkum merged 1 commit intodevelopfrom
fix-PB-859-doubling-length

Conversation

@ltkum
Copy link
Contributor

@ltkum ltkum commented Sep 17, 2024

All geometries were interpreted as polygons, meaning a line was counted back and forth (doubling the length...)

Could not reliably add a Cypress test to grab the tooltip on the map (the one in red) as it is drawn directly in the canvas (no HTML element to grab with a CSS selector...)

Test link

@ltkum ltkum requested a review from ltshb September 17, 2024 08:01
@github-actions github-actions bot added the bug label Sep 17, 2024
All geometries were interpreted as polygons, meaning a line was counted back and forth (doubling the length...)
@ltkum ltkum force-pushed the fix-PB-859-doubling-length branch from c0b418c to ab84cd5 Compare September 17, 2024 08:01
@ltshb ltshb changed the title PB-859 : fix doubling of length value PB-1007: fix doubling of length value Sep 17, 2024
@cypress
Copy link

cypress bot commented Sep 17, 2024

web-mapviewer    Run #3328

Run Properties:  status check passed Passed #3328  •  git commit ab84cd5beb: PB-859 : fix doubling of length value
Project web-mapviewer
Branch Review fix-PB-859-doubling-length
Run status status check passed Passed #3328
Run duration 04m 54s
Commit git commit ab84cd5beb: PB-859 : fix doubling of length value
Committer Martin Künzi
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 210
View all changes introduced in this branch ↗︎

@ltkum ltkum merged commit 228cdf8 into develop Sep 17, 2024
@ltkum ltkum deleted the fix-PB-859-doubling-length branch September 17, 2024 08:26
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.

2 participants