Skip to content

Conversation

@mahmoudadel54
Copy link
Contributor

Description

This PR includes fix issue of using coordinate format from mapInfo state within Share Panel plugin by creating a separate local state for Share panel only without any dependent with Identify.

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

What is the current behavior?
#11777

What is the new behavior?

In share panel, the coordinate editor reads the init value of coordinate format from localConfig or decimal by default and not related with the Identify formatCoord at all.

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

…ool depend on each other instead of the config in localConfig.json

Descrption:
- create local state for format coordinate in Share plugin instead of using mapInfo one
- add unit test
@mahmoudadel54 mahmoudadel54 added this to the 2025.02.01 milestone Dec 5, 2025
@mahmoudadel54 mahmoudadel54 self-assigned this Dec 5, 2025
@tdipisa tdipisa requested review from MV88 and removed request for allyoucanmap December 5, 2025 14:25
@tdipisa tdipisa assigned MV88 and mahmoudadel54 and unassigned mahmoudadel54 Dec 5, 2025
@MV88 MV88 merged commit f61c402 into geosolutions-it:master Dec 5, 2025
11 checks passed
@MV88
Copy link
Contributor

MV88 commented Dec 5, 2025

@ElenaGallo please test in DEV

github-actions bot pushed a commit that referenced this pull request Dec 12, 2025
…ach other instead of the config in localConfig.json (#11778)

Descrption:
- create local state for format coordinate in Share plugin instead of using mapInfo one
- add unit test

(cherry picked from commit f61c402)
@github-actions
Copy link
Contributor

@ElenaGallo
Copy link
Contributor

Test passed, @tdipisa backport can be merged

tdipisa pushed a commit that referenced this pull request Dec 12, 2025
…ach other instead of the config in localConfig.json (#11778) (#11810)

Descrption:
- create local state for format coordinate in Share plugin instead of using mapInfo one
- add unit test

(cherry picked from commit f61c402)

Co-authored-by: mahmoud adel <58145645+mahmoudadel54@users.noreply.github.com>
mahesh-wor pushed a commit to mahesh-wor/MapStore2 that referenced this pull request Dec 17, 2025
…ool depend on each other instead of the config in localConfig.json (geosolutions-it#11778)

Descrption:
- create local state for format coordinate in Share plugin instead of using mapInfo one
- add unit test
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.

Coordinate in the GFI and Share-tool depend on each other instead of the config in localConfig.json

4 participants