Skip to content

#12593 Improving Identify UX making some parameters configurable from UI#12609

Open
Binabh wants to merge 4 commits into
geosolutions-it:masterfrom
Binabh:feat/add-feature-count-and-buffer
Open

#12593 Improving Identify UX making some parameters configurable from UI#12609
Binabh wants to merge 4 commits into
geosolutions-it:masterfrom
Binabh:feat/add-feature-count-and-buffer

Conversation

@Binabh

@Binabh Binabh commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

  • Added FeatureInfoRequestOptions component to manage feature info settings.
  • Added WMS layer handling to preserve feature info configurations.
  • Added tests to validate new feature info options and their behavior.
  • Added translations for feature info request options in multiple languages.

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Feature

Issue

What is the current behavior?

#12593

What is the new behavior?
Inputs have been added to catalog advanced settings for WMS layers and in layer settings. User can now input values there and that will be sent in GetFeatureInfo request.

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

- Added FeatureInfoRequestOptions component to manage feature info settings.
- Added WMS layer handling to preserve feature info configurations.
- Added tests to validate new feature info options and their behavior.
- Added translations for feature info request options in multiple languages.
@Binabh Binabh added this to the 2026.03.00 milestone Jul 9, 2026
@Binabh
Binabh requested a review from allyoucanmap July 9, 2026 11:00
@cla-bot cla-bot Bot added the CLA Ready label Jul 9, 2026
@Binabh Binabh linked an issue Jul 9, 2026 that may be closed by this pull request
6 tasks
@offtherailz

Copy link
Copy Markdown
Member

@Binabh I updated the master (your previous merge look to have been done before fixing the #12605 was merged) 🤞

@allyoucanmap
allyoucanmap requested review from stefanocudini and removed request for allyoucanmap July 14, 2026 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improving Identify UX making some parameters configurable from UI

5 participants