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

#10544: fix Embedded maps - details loads panel even if the details attribute value is NODATA #10548

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

mahmoudadel54
Copy link
Contributor

Description

In this PR, adding check of no data condition for show/hide details panel in toolbar has been implemented to fix the issue.

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

#10544

What is the current behavior?
#10544

What is the new behavior?
The details panel is hidden in the toolbar for embeded maps if the details attribute value is NODATA.

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

…f the details attribute value is NODATA

Description:
- adding check of no data condition for show/hide details panel in toolbar
@MV88 MV88 added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 16, 2024
@MV88 MV88 merged commit c4c5d4b into geosolutions-it:master Sep 16, 2024
6 checks passed
@MV88
Copy link
Contributor

MV88 commented Sep 16, 2024

@ElenaGallo please test it in DEV and let us know if it possible to backport it to 2024.02.xx

@ElenaGallo
Copy link
Contributor

Test passed, @mahmoudadel54 please backport it to 2024.02.xx, thanks

mahmoudadel54 added a commit to mahmoudadel54/MapStore2 that referenced this pull request Sep 18, 2024
…f the details attribute value is NODATA (geosolutions-it#10548)

Description:
- adding check of no data condition for show/hide details panel in toolbar
@mahmoudadel54
Copy link
Contributor Author

Test passed, @mahmoudadel54 please backport it to 2024.02.xx, thanks

Backport is done --- > #10558

tdipisa pushed a commit that referenced this pull request Sep 19, 2024
…ttribute value is NODATA (#10548) (#10558)

Description:
- adding check of no data condition for show/hide details panel in toolbar
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 19, 2024
offtherailz pushed a commit that referenced this pull request Oct 18, 2024
…ttribute value is NODATA (#10548)

Description:
- adding check of no data condition for show/hide details panel in toolbar
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.

Embedded maps - details loads panel even if the details attribute value is NODATA
4 participants