Skip to content

fix: /tile request should be with verbose = true#321

Merged
nilsnolde merged 2 commits intomasterfrom
nn-tile-verbose
Jan 29, 2026
Merged

fix: /tile request should be with verbose = true#321
nilsnolde merged 2 commits intomasterfrom
nn-tile-verbose

Conversation

@nilsnolde
Copy link
Member

@nilsnolde nilsnolde commented Jan 29, 2026

🛠️ Fixes Issue

fixes #320

currently https://valhalla.openstreetmap.de/tiles?profile=bicycle&style=carto shows only 2 edge attributes.

@ghost
Copy link

ghost commented Jan 29, 2026

Preview is ready! 🚀 You can view it here: https://valhalla-app-tests.gis-ops.com/321

@nilsnolde nilsnolde requested a review from chrstnbwnkl January 29, 2026 19:23
mustaphaturhan
mustaphaturhan previously approved these changes Jan 29, 2026
First install the `playwright` browsers:

```bash
npx playwright install
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

--with-deps was causing some error. arch linux doesn't seem to be natively supported for playwright, it's installing the fallback ubuntu browsers. for some reason it only worked without that argument

@nilsnolde
Copy link
Member Author

@mustaphaturhan could you re-approve, I didn't run the tests at first..

@nilsnolde
Copy link
Member Author

actually never mind;)

@nilsnolde nilsnolde merged commit 0b01546 into master Jan 29, 2026
3 checks passed
@nilsnolde nilsnolde deleted the nn-tile-verbose branch January 29, 2026 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add "verbose": true to the /tile request

2 participants