-
Notifications
You must be signed in to change notification settings - Fork 9
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
Create stock overpass query using drawn shape #18
Conversation
8183a8f
to
3bcfb20
Compare
|
- add some utility functions which can send a geojson polygon to the overpass api, along with a stock request for building data - update map component to display building data by point
3bcfb20
to
ea425ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 tested and works in both IE11 and Chrome. Not in this PR, but we'll need to refine this workflow somewhat. I think it's more natural for us to fetch the values and show them on screen before the user clicks "extract" to download the shapefile, but given the delay of the fetch it makes sense to have a button that starts that too.
Sounds good to me. I'll tweak it a little while working on #8 |
Thanks for reviewing! |
Overview
This PR creates a stock overpass query using the drawn shape sent by the user.
The workflow for this is currently not the most apparent, but we can fix that as we go.
Connects #6
Demo
Notes
We're going to have to make this Overpass API request way more configurable but it's better to push off handling that until we have the configuration for the actual set of features.
Testing Instructions
update
andserver