Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#10026: fix functional tests for Interactive legend for TOC layers [WMS Support] #10249

Merged
merged 4 commits into from
Apr 24, 2024

Conversation

mahmoudadel54
Copy link
Collaborator

Description

This PR includes fixing functional tests for interactive legend #10026:

  • Hide checkbox of enabling interactive legend for WMS layers in dashboard and geostory as they will be implemented in a separate issue.
  • fix not saving the flag of enabling interactive legend into wms layers in case save map.

Please check if the PR fulfills these requirements

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

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

#10026

What is the current behavior?
Saving map with a layer that has a flag of enabling interactive legend to keep it in each open this map was not working well.
Dashboard and geostory were containing the checkbox of enabling interactive legend in Display tab in layer settings.

What is the new behavior?
Save map is handling storing the enable interactive legend flag to keep it stored if user close the map and open it again.
For dashboard and geostory pages, the enable interactive legend checkbox in layer settings in Display tab is hidden now.

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

Other useful information

…port]

Description:
- fix functional tests for interactive legend geosolutions-it#10026
@mahmoudadel54 mahmoudadel54 added this to the 2024.02.00 milestone Apr 23, 2024
@mahmoudadel54 mahmoudadel54 self-assigned this Apr 23, 2024
@mahmoudadel54 mahmoudadel54 linked an issue Apr 23, 2024 that may be closed by this pull request
6 tasks
…port]

Description:
- fix import mistake in TOCItemsSettings file
web/client/components/TOC/fragments/settings/Display.jsx Outdated Show resolved Hide resolved
web/client/plugins/TOCItemsSettings.jsx Outdated Show resolved Hide resolved
web/client/selectors/map.js Outdated Show resolved Hide resolved
…port]

Description:
- resolve review comments by adding a configration prop for TOCItemsSettings 'hideInteractiveLegendOption'
- remove the replaced selector 'isMapOnlyOpenedSelector'
- edit unit test of 'Display-test' file
-
…port]

Description:
- revert the change in map.js in selectors
@allyoucanmap allyoucanmap merged commit 67252e2 into geosolutions-it:master Apr 24, 2024
6 checks passed
@allyoucanmap
Copy link
Contributor

allyoucanmap commented Apr 24, 2024

@ElenaGallo please test on dev and let us know for the backport, thanks.

@ElenaGallo
Copy link
Contributor

Test passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 participants