Skip to content

Fix #3706 Add support for simple GeoJSON to shapefile#3707

Merged
MV88 merged 4 commits intogeosolutions-it:masterfrom
MV88:3706_support_simple_geojson
May 2, 2019
Merged

Fix #3706 Add support for simple GeoJSON to shapefile#3707
MV88 merged 4 commits intogeosolutions-it:masterfrom
MV88:3706_support_simple_geojson

Conversation

@MV88
Copy link
Copy Markdown
Contributor

@MV88 MV88 commented Apr 30, 2019

Description

added support for simple geoJON files (.json) contained in .zip

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)
you cannot drop zip file containig .json files with simple geojson feature

What is the new behavior?
you can

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

  • Yes, and I documented them in migration notes
  • No

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

Other information:

@MV88 MV88 added this to the 2019.02.00 milestone Apr 30, 2019
@MV88 MV88 requested a review from mbarto April 30, 2019 12:22
@MV88 MV88 self-assigned this Apr 30, 2019
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 30, 2019

Coverage Status

Coverage increased (+0.1%) to 81.366% when pulling ca0ccd0 on MV88:3706_support_simple_geojson into 1772771 on geosolutions-it:master.

@mbarto
Copy link
Copy Markdown
Contributor

mbarto commented Apr 30, 2019

Fix tests, travis build is failing

@MV88 MV88 merged commit 486fb85 into geosolutions-it:master May 2, 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.

Add support for simple GeoJSON features

3 participants