search

gis-ops / valhalla-app

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

Add warnings from Valhalla #45

Open nilsnolde opened 1 year ago

nilsnolde commented 1 year ago

Since we implemented a warnings array for user feedback, we should find a way to display them. E.g. one of the most important information it returns is when we clamp the value of an input costing option to a default value when it's out of bounds, e.g. top_speed can only be 10 - 252 km/h and if it exceeds that range we clamp it to 140. That's important info for the user.

theWiseAman commented 1 year ago

@nilsnolde Can you assign this issue to me? I investigated a little bit and I found where I need to make/add changes. I am already familiar with ReactJS so I have some ideas of what can be done here. Can I straightaway create a pull request?

theWiseAman commented 1 year ago

Since we implemented a warnings array for user feedback, we should find a way to display them.

Can you provide some more context around it as it seems I am unable to find any such array using all file search results in vs code?

ganesh1828 commented 1 year ago

@theWiseAman are you still working on this issue ? or can I take this issue ?

Ruthvik2707 commented 8 months ago

@nilsnolde Hey, I just stumbled on the issues and I would like to work on them, Can you assign the issue in the meantime I will work on it if I can get the solution and some elaboration on the issue and its fixes.

nilsnolde commented 8 months ago

Just PR when smth is ready, thanks.

man0045 commented 7 months ago

can you please assign me this issue, i love to contribute in this issue