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

turn shapefiles into geojson in browser #10

Closed
mapsam opened this issue Mar 25, 2015 · 11 comments · Fixed by #192
Closed

turn shapefiles into geojson in browser #10

mapsam opened this issue Mar 25, 2015 · 11 comments · Fixed by #192

Comments

@mapsam
Copy link
Member

mapsam commented Mar 25, 2015

using https://github.com/calvinmetcalf/shapefile-js

@daguar
Copy link

daguar commented Mar 25, 2015

👍

@mapsam
Copy link
Member Author

mapsam commented Apr 24, 2015

Should probably just use https://github.com/mapbox/leaflet-omnivore to cover all of our bases quickly

@mapsam mapsam modified the milestone: Pre-alpha (Spring Fling) Apr 28, 2015
@nickpeihl
Copy link
Member

👍

Very few of my users have ever heard of geojson. Shapefiles are well-known though.

@mapsam
Copy link
Member Author

mapsam commented May 7, 2015

I agree. I think it's unlikely that people will have GeoJSON files lying around for them to edit. I'll make sure this becomes a priority during our sprint next week!

@nickpeihl
Copy link
Member

Cool. Looking forward to meeting you at the Spring Fling.

@mapsam
Copy link
Member Author

mapsam commented May 7, 2015

Likewise! 💃

@lyzidiamond
Copy link
Contributor

@lyzidiamond
Copy link
Contributor

Oh snaps I should read comments before I post them... sorry to be repetitive 😨

@nickpeihl
Copy link
Member

Unless I'm missing something, leaflet-omnivore doesn't appear to handle shapefiles. Perhaps leaflet-shapefile would work.

@mapsam
Copy link
Member Author

mapsam commented May 14, 2015

@npeihl you are correct. I think the shapefile piece will have to be different but including omnivore is going to be part of the same process.

@aaronr what was your work-around for using modules without require()? Would it be worth adding browserify #8 so we can get these issues out of the way?

@mapsam
Copy link
Member Author

mapsam commented Oct 16, 2015

Thanks to @jczaplew for knocking this one out!

dropchop-loadshapefile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants