Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KnownTileSources: update Stamen URLs #200

Merged
merged 2 commits into from
Jan 18, 2023
Merged

Conversation

janusw
Copy link
Member

@janusw janusw commented Jan 14, 2023

  • use https URLs with new hostnames, as documented on maps.stamen.com
  • the set of available subdomains still seems to be the same ('a' to 'd')
  • note: the toner-lite layer is not documented on maps.stamen.com explicitly, but it still seems to work

@janusw
Copy link
Member Author

janusw commented Jan 14, 2023

Here is an example URL that I used for testing (with variations):
https://stamen-tiles-d.a.ssl.fastly.net/terrain/12/654/1583.png

@janusw
Copy link
Member Author

janusw commented Jan 14, 2023

Should fix #199, but I haven't verified that yet (will be much easier for me once #198 is merged).

* use https URLs with new hostnames, as documented on maps.stamen.com
* the set of available subdomains still seems to be the same ('a' to 'd')
* note: the toner-lite layer is not documented on maps.stamen.com explicitly,
  but it still seems to work
@janusw
Copy link
Member Author

janusw commented Jan 18, 2023

I just rebased this to current master, in order to take advantage of #198. Will do some tests soon.

@janusw
Copy link
Member Author

janusw commented Jan 18, 2023

Alright, I tried the nupkg from https://github.com/BruTile/BruTile/actions/runs/3948252149#artifacts and can confirm that it works nicely and enables me to display the Stamen Terrain map on iOS and Android. I.e. this PR does fix #199 for me 🥳

Copy link
Contributor

@pauldendulk pauldendulk left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this!

@pauldendulk pauldendulk merged commit 496a86d into BruTile:master Jan 18, 2023
@janusw janusw deleted the stamen branch January 18, 2023 13:13
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.

None yet

2 participants