Skip to content

Fix #3682 request to printUrl#3683

Merged
MV88 merged 3 commits intogeosolutions-it:masterfrom
MV88:3682_print_fixes
Apr 9, 2019
Merged

Fix #3682 request to printUrl#3683
MV88 merged 3 commits intogeosolutions-it:masterfrom
MV88:3682_print_fixes

Conversation

@MV88
Copy link
Copy Markdown
Contributor

@MV88 MV88 commented Apr 9, 2019

Description

This pr add the possibiltty to disable request for printUrl

it also removes a warning from the Toolbars

Issues

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:

What is the current behavior? (You can also link to an open issue here)
it perform request to printUrl defined in localconfig

What is the new behavior?
it no longer does if properly configured

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.03%) to 81.027% when pulling 78ebf4f on MV88:3682_print_fixes into 19ddd55 on geosolutions-it:master.

@MV88 MV88 merged commit 716b6c7 into geosolutions-it:master Apr 9, 2019
@MV88 MV88 removed the 1 Point label Apr 11, 2019
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.

avoid request to printUrl if print plugin is absent

3 participants