Skip to content

PB-1211 : do not add ; if layer with feature selection is alone#1140

Merged
pakb merged 1 commit intodevelopfrom
bug-PB-1211-more-link-issue
Nov 26, 2024
Merged

PB-1211 : do not add ; if layer with feature selection is alone#1140
pakb merged 1 commit intodevelopfrom
bug-PB-1211-more-link-issue

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Nov 26, 2024

When opening the app with a layerId=featureId URL param, the ; was still added to the layers URL param after processing the feature pre-selection. This means the layers URL param was malformed and an error showed up.

Test link

When opening the app with a layerId=featureId URL param, the ; was still added to the layers URL param after processing the feature pre-selection. This means the layers URL param was malformed and an error showed up.
@pakb pakb requested a review from ltkum November 26, 2024 12:31
@github-actions github-actions bot added the bug label Nov 26, 2024
@cypress
Copy link

cypress bot commented Nov 26, 2024

web-mapviewer    Run #3907

Run Properties:  status check passed Passed #3907  •  git commit 4e091ae943: PB-1211 : do not add ; if layer with feature selection is alone
Project web-mapviewer
Branch Review bug-PB-1211-more-link-issue
Run status status check passed Passed #3907
Run duration 05m 27s
Commit git commit 4e091ae943: PB-1211 : do not add ; if layer with feature selection is alone
Committer Pascal Barth
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 212
View all changes introduced in this branch ↗︎

@pakb pakb merged commit f501424 into develop Nov 26, 2024
@pakb pakb deleted the bug-PB-1211-more-link-issue branch November 26, 2024 12:43
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