Skip to content

Fix #4758 Ability to save resource with thumbnail not allowed#5760

Merged
offtherailz merged 1 commit intogeosolutions-it:masterfrom
vlt1:fix-4758
Aug 10, 2020
Merged

Fix #4758 Ability to save resource with thumbnail not allowed#5760
offtherailz merged 1 commit intogeosolutions-it:masterfrom
vlt1:fix-4758

Conversation

@vlt1
Copy link
Copy Markdown
Contributor

@vlt1 vlt1 commented Aug 7, 2020

Description

Resolves incorrect handling of NODATA is some cases for linked resources.

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

#4758

What is the current behavior?

#4758

What is the new behavior?
Thumbnail is always in the correct state.

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

@vlt1 vlt1 added this to the 2020.02.01 milestone Aug 7, 2020
@vlt1 vlt1 self-assigned this Aug 7, 2020
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 83.467% when pulling 79c1b22 on vlt1:fix-4758 into 677ceb2 on geosolutions-it:master.

@offtherailz offtherailz merged commit f1b1326 into geosolutions-it:master Aug 10, 2020
@offtherailz
Copy link
Copy Markdown
Member

it's ok, thank you 👍 please backport this to 2020.02.xx

@offtherailz offtherailz added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Aug 10, 2020
vlt1 added a commit to vlt1/MapStore2 that referenced this pull request Aug 12, 2020
offtherailz pushed a commit that referenced this pull request Aug 12, 2020
@offtherailz offtherailz removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Aug 12, 2020
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.

4 participants