Skip to content

Conversation

@sommerfe
Copy link
Contributor

@sommerfe sommerfe commented Jan 13, 2026

@sommerfe sommerfe self-assigned this Jan 13, 2026
@github-actions github-actions bot added the bug label Jan 13, 2026
@sommerfe sommerfe force-pushed the fix-pb-2064-drawing-module branch 2 times, most recently from 2aab2c9 to 2763530 Compare January 14, 2026 14:14
@sommerfe sommerfe requested review from ismailsunni and pakb January 14, 2026 14:28
Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

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

Looks like you have based your work on #1546 you should switch the base of your PR

Comment on lines +770 to +771
// Register proj4 with OpenLayers
register(proj4)
Copy link
Contributor

@pakb pakb Jan 15, 2026

Choose a reason for hiding this comment

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

I had mixed result with this, calling OL register inside a bundled lib.
It looked like the proj4 instance we are altering here can be different from the one OL then receives in the overlying app (the one importing this piece of code).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you have a suggestion on how i can do it differently? the problem is that the EPSG:2056 is missing when calling the parseKML function from the kmlUtils. So the drawing is not visible as the KML coordinates can not be converted.

@sommerfe sommerfe force-pushed the fix-pb-2064-drawing-module branch from 2763530 to 393a060 Compare January 15, 2026 07:33
@sommerfe sommerfe changed the base branch from feat-PB-1383-pinia-store to fix-pb-2064-import-tool-file January 15, 2026 07:35
@sommerfe sommerfe changed the base branch from fix-pb-2064-import-tool-file to fix-pb-2026-fix-import-tool-file January 15, 2026 07:35
@sommerfe sommerfe changed the base branch from fix-pb-2026-fix-import-tool-file to feat-PB-1383-pinia-store January 15, 2026 07:36
@sommerfe sommerfe force-pushed the fix-pb-2064-drawing-module branch from 393a060 to c2de108 Compare January 15, 2026 07:48
@sommerfe sommerfe requested a review from pakb January 15, 2026 08:05
@cypress
Copy link

cypress bot commented Jan 15, 2026

web-mapviewer    Run #6369

Run Properties:  status check failed Failed #6369  •  git commit 457a1378d4: PB-2064: fix lint
Project web-mapviewer
Branch Review fix-pb-2064-drawing-module
Run status status check failed Failed #6369
Run duration 19m 27s
Commit git commit 457a1378d4: PB-2064: fix lint
Committer Felix Sommer
View all properties for this run ↗︎

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

Tests for review

Failed  drawing.cy.ts • 14 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Drawing module tests > Drawing mode/tools > can create marker/icons and edit them Test Replay Screenshots
Drawing module tests > Drawing mode/tools > can create annotation/text and edit them Test Replay Screenshots
Drawing module tests > Drawing mode/tools > can create line / measurement, extend it, and delete the last node by right click / button, and make a polygon Test Replay Screenshots
Drawing module tests > Drawing mode/tools > can create line/polygons and edit them Test Replay Screenshots
Drawing module tests > KML management > deletes the drawing when confirming the delete modal Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer in the layer list / URL params correctly Test Replay Screenshots
Drawing module tests > KML management > keeps the KML after a page reload, and creates a copy if it is then edited Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer correctly if it comes attached with an adminId at startup Test Replay Screenshots
Drawing module tests > KML management > manages the KML layer correctly if it comes attached with an adminId at startup from a legacy URL Test Replay Screenshots
Drawing module tests > KML management > receives an empty KML and can use drawing mode Test Replay Screenshots
The first 10 failed tests are shown, see all 14 tests in Cypress Cloud.
Failed  legacyParamImport.cy.ts • 6 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Test on legacy param import > Coordinates import > center where expected when given a X, Y coordinate in LV03 Test Replay Screenshots
Test on legacy param import > Layers import > is able to import an external KML from a legacy adminId query param Test Replay Screenshots
Test on legacy param import > Layers import > don't keep KML adminId in URL after import Test Replay Screenshots
Test on legacy param import > Layers import > is able to import an external KML from a legacy adminId query param with other layers Test Replay Screenshots
Test on legacy param import > 3D import > transfers camera parameter from legacy URL to the new URL only heading Test Replay Screenshots
... > Select a few features and shows the tooltip in its correct spot Test Replay Screenshots
Failed  3d/layers.cy.ts • 10 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Test of layer handling in 3D > add layer from search bar Test Replay Screenshots
Test of layer handling in 3D > sets the opacity to the value defined in the layers URL param or menu UI Test Replay Screenshots
Test of layer handling in 3D > sets the timestamp of a layer when specified in the layers URL param Test Replay Screenshots
Test of layer handling in 3D > reorders visible layers when corresponding buttons are pressed Test Replay Screenshots
Test of layer handling in 3D > add GeoJson layer with opacity from URL param Test Replay Screenshots
Test of layer handling in 3D > removes a layer from the visible layers when the "remove" button is pressed Test Replay Screenshots
Test of layer handling in 3D > uses the 3D configuration of a layer if one exists Test Replay Screenshots
Test of layer handling in 3D > add KML layer from drawing Test Replay Screenshots
Test of layer handling in 3D > Verify layer features in 2D and 3D Test Replay Screenshots
Test of layer handling in 3D > Verify a layer with EPSG:4326(WEBMERCATOR) bounding box in 2D and 3D Test Replay Screenshots
Failed  featureSelection.cy.ts • 4 failed tests • e2e/chrome/mobile

View Output

Test Artifacts
Testing the feature selection > Feature pre-selection in the URL > Synchronise URL and feature selection Test Replay Screenshots
Testing the feature selection > Feature identification on the map > can select an area to identify features inside it Test Replay Screenshots
Testing the feature selection > Feature identification on the map > can select feature by click, add more feature, and deselect feature Test Replay Screenshots
Testing the feature selection > Feature identification on the map > can print feature information Test Replay Screenshots
Failed  importToolMaps.cy.ts • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
The Import Maps Tool > Import external WMTS layers Test Replay Screenshots

The first 5 failed specs are shown, see all 15 specs in Cypress Cloud.

@sommerfe sommerfe force-pushed the fix-pb-2064-drawing-module branch from ecac3a6 to 457a137 Compare January 15, 2026 12:37
@sommerfe sommerfe requested a review from schtibe January 16, 2026 08:49
@sommerfe sommerfe merged commit 4235c99 into feat-PB-1383-pinia-store Jan 16, 2026
4 of 6 checks passed
@sommerfe sommerfe deleted the fix-pb-2064-drawing-module branch January 16, 2026 11:08
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.

3 participants