search

gis-ops / valhalla-app

This is the demo web app running on https://valhalla.openstreetmap.de
https://valhalla.openstreetmap.de
MIT License
164 stars 90 forks source link

Route summary reporting error fixed #62

Closed rakeshhotker closed 1 year ago

rakeshhotker commented 1 year ago

changed the code for formatDuration function defined in Summary.jsx to be same as the one defined in Map.jsx. As the definition in Map.jsx seemed more appropriate. Do let me know if any modifications are required. Update: Also did the enhancement as required of showing the user the tileset age. Do let me know if any modifications are required.

nilsnolde commented 1 year ago

You did not mention this is for GSoC, see #67 . We can open it again once we get to this point.

nilsnolde commented 1 year ago

Now that we have some guidelines, I'll open this again. https://github.com/gis-ops/valhalla-app/wiki/GSoC-2023

nilsnolde commented 1 year ago

I'd just ask you to revert the tileset age thing and focus on just one issue.