Skip to content

Add Shortbread style JSON URLs#151

Draft
gy-mate wants to merge 4 commits intoopenstreetmap:mainfrom
gy-mate:add-style-json
Draft

Add Shortbread style JSON URLs#151
gy-mate wants to merge 4 commits intoopenstreetmap:mainfrom
gy-mate:add-style-json

Conversation

@gy-mate
Copy link

@gy-mate gy-mate commented Jul 23, 2025

In order to make setting it up in QGIS (for personal use) easier.

Also:

  • change the zoom variable to z as required by QGIS
  • fix grammar.

@Firefishy Firefishy requested a review from pnorman July 23, 2025 12:24
Copy link
Collaborator

@pnorman pnorman left a comment

Choose a reason for hiding this comment

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

I'm reluctant to imply long-term support for the internal style URL. It commits us to continuing to host the specific style long term while we've only done that for the vector tiles.

Also, the style URL is wrong - it's not $VERSION (e.g. shortbread_v1), it's the style name.

If we do publish the style URL it should be a separate paragraph and recommend people host the style themselves. That way they won't get surprised by style changes that don't work with whatever they're doing with the map.

@gy-mate gy-mate requested a review from pnorman August 17, 2025 16:20
@gy-mate
Copy link
Author

gy-mate commented Aug 17, 2025

That's a good point!

Is it okay this way?

@pnorman
Copy link
Collaborator

pnorman commented Aug 17, 2025

That's a good point!

Is it okay this way?

We still haven't decided on a policy in openstreetmap/operations#1263. Once that is decided we'll know what the URLs are, if any.

@pnorman pnorman marked this pull request as draft August 17, 2025 22:54
@gy-mate
Copy link
Author

gy-mate commented Aug 18, 2025

Sure, thanks! :)

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.

2 participants