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

#9589: Save layer sourceMetadata info #9736

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

dsuren1
Copy link
Contributor

@dsuren1 dsuren1 commented Nov 23, 2023

Description

This PR fixes the layer's sourceMetadata not saved when exporting or saving the 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

Issue

What is the current behavior?

What is the new behavior?
The layer's sourceMetadata is saved properly on export and map save

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

Note for testers

Kindly DO NOT use the old map import. Add a layer from COG with metadata fetched and export it. Then try importing the same to an instance that has projectionDefs configured to test this bug and to an instance where projectionDefs are not configured to check if notification works fine

@dsuren1 dsuren1 added bug BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch labels Nov 23, 2023
@dsuren1 dsuren1 added this to the 2023.02.01 milestone Nov 23, 2023
@dsuren1 dsuren1 self-assigned this Nov 23, 2023
@dsuren1 dsuren1 linked an issue Nov 23, 2023 that may be closed by this pull request
6 tasks
@offtherailz offtherailz merged commit 170bce7 into geosolutions-it:master Nov 27, 2023
6 checks passed
@offtherailz
Copy link
Member

@ElenaGallo, could you please test this on DEV ? Thank you

@tdipisa
Copy link
Member

tdipisa commented Nov 27, 2023

@ElenaGallo let us know if we can backport

@ElenaGallo
Copy link
Contributor

Test passed, @dsuren1 please backport this. Thanks

dsuren1 added a commit to dsuren1/MapStore2 that referenced this pull request Nov 29, 2023
tdipisa pushed a commit that referenced this pull request Nov 29, 2023
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 29, 2023
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.

Notify data projection not compatible / available for COG
4 participants