Kevin Kreiser
Kevin Kreiser
ah i completely misunderstood what he was saying. i had no idea we had geojson export on the demo server front end. apologies for the confusion
@Sleepingwell this is looking good thanks for all of the effort here. i left a few comments so far, let me know what you think or if they dont make...
@Sleepingwell this basically looks done to me, save for some tiny little stuff. sorry i didnt look at it for so long. do you mind if i set it to...
@Sleepingwell all 3 excellent points and definitely the next steps. let me see if i can describe what needs to be done: 1. updating the docs should be straight forward...
> Should we add the OSM id for the node at the start of an edge? this isnt a bad idea but we would need some way to also denote...
@devShahriar as you can see we havent merged the pr so it depends what you mean by "ready". you can definitely checkout @Sleepingwell 's branch and try it yourself. read...
also @Sleepingwell it looks like you need to lint the code (format it) by running `scripts/format.sh`
@Sleepingwell we only allow squash merges to the repo anyway so it will all become one commit in the end anyway. unless you really want to there is really no...
i havent gotten a chance to look deeply at this yet, i just wanted to apologize for the wait
Yep I totally agree. Let me put this on my schedule for Monday morning