Skip to content

PB-1220: Allow SLD enabled WMS to display aggregate legends#1369

Merged
pakb merged 2 commits intodevelopfrom
fix-PB-1220-allow-sld-legends-parsing
Jul 2, 2025
Merged

PB-1220: Allow SLD enabled WMS to display aggregate legends#1369
pakb merged 2 commits intodevelopfrom
fix-PB-1220-allow-sld-legends-parsing

Conversation

@ltkum
Copy link
Contributor

@ltkum ltkum commented Jun 30, 2025

Issue: In some wms, there exists legends for multiple sub layers, which are available through a SLD-enabled endpoint. We would like to show them
Fix: We parse SLD attributes and add legends where there is no `style` element for it, and the `getLegendGraphic` endpoint is enabled
@github-actions github-actions bot added the bug label Jun 30, 2025
@cypress
Copy link

cypress bot commented Jun 30, 2025

web-mapviewer    Run #5487

Run Properties:  status check passed Passed #5487  •  git commit c345078eff: PB-1220: Unit Tests and E2E tests
Project web-mapviewer
Branch Review fix-PB-1220-allow-sld-legends-parsing
Run status status check passed Passed #5487
Run duration 05m 25s
Commit git commit c345078eff: PB-1220: Unit Tests and E2E tests
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 20
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 253
View all changes introduced in this branch ↗︎

@ltkum ltkum force-pushed the fix-PB-1220-allow-sld-legends-parsing branch 3 times, most recently from 87f1b8a to fe083b9 Compare June 30, 2025 15:14
@ltkum ltkum force-pushed the fix-PB-1220-allow-sld-legends-parsing branch from fe083b9 to c345078 Compare June 30, 2025 15:39
@ltkum ltkum requested a review from pakb June 30, 2025 15:57
@pakb pakb marked this pull request as ready for review July 1, 2025 07:31
@pakb pakb changed the title Fix pb 1220 allow sld legends parsing PB-1220: allow WMS SLD legends parsing Jul 2, 2025
@pakb pakb changed the title PB-1220: allow WMS SLD legends parsing PB-1220: Allow SLD enabled WMS to display aggregate legends Jul 2, 2025
@pakb pakb merged commit 0aceedf into develop Jul 2, 2025
12 checks passed
@pakb pakb deleted the fix-PB-1220-allow-sld-legends-parsing branch July 2, 2025 08:35
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