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

#10622: Fix Not possible to remove widget from Map - JavaScript error #10623

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

mahmoudadel54
Copy link
Contributor

@mahmoudadel54 mahmoudadel54 commented Oct 23, 2024

Description

This PR fixes the issue of not remove widget from map in case of existing other widgets has dependenciesMap prop = empty object.

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

#10622

What is the current behavior?
#10622

What is the new behavior?
The delete confirmation dialog window closes and the widget is removed from the map normally.

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

…cript error

Description:
- fix the issue of not remove widget from map in case of existing other widgets has dependenciesMap prop = empty object
- add unit test
Copy link
Contributor

@MV88 MV88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MV88 MV88 merged commit 3794608 into geosolutions-it:master Oct 24, 2024
6 checks passed
@MV88
Copy link
Contributor

MV88 commented Oct 24, 2024

@ElenaGallo pls test it in DEV

@ElenaGallo ElenaGallo added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Oct 25, 2024
@ElenaGallo
Copy link
Contributor

ElenaGallo commented Oct 25, 2024

Test passed, @mahmoudadel54 please backport to 2024.02.xx. Thanks

@tdipisa
Copy link
Member

tdipisa commented Oct 25, 2024

Test passed, @mahmoudadel54 please backport to 2024.02.xx. Thanks

@mahmoudadel54 the PR need to be also backported to c027-2024-10-24 branch. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch bug C027-COMUNE_FI-2023-SUPPORT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not possible to remove widget from Map - JavaScript error
4 participants