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

added Stadia Maps to list of providers #344

Merged
merged 1 commit into from
Jan 30, 2020

Conversation

holtalanm
Copy link

@holtalanm holtalanm commented Jan 27, 2020

@brunob
Copy link
Member

brunob commented Jan 28, 2020

@jieter any thoughts ?

@jieter
Copy link
Contributor

jieter commented Jan 30, 2020

LGTM

@brunob brunob merged commit 8627840 into leaflet-extras:master Jan 30, 2020
@brunob
Copy link
Member

brunob commented Jan 30, 2020

Thx @holtalanm :)

@jieter
Copy link
Contributor

jieter commented Jan 30, 2020

Hmm, there is no preview API key configured, so http://leaflet-extras.github.io/leaflet-providers/preview/#filter=Stadia shows grey maps.

We should either keep it out of the preview, or add an example API code:

var exampleAPIcodes = {
'HERE': {
'app_id': 'tFZyfnyJAmhfh5gdoGcR',
'app_code': 'vJ8o9OCQ1o0Y2wwbRspzSA'
},
'Thunderforest': {
'apikey': 'db5ae1f5778a448ca662554581f283c5'
}
};

@brunob
Copy link
Member

brunob commented Jan 30, 2020

As what i can read in their doc you don't need an API key to use the tiles, but the error is 403 Forbidden Request forbidden by administrative rules. , and i've seen this one while testing their static map API ref https://seenthis.net/messages/822843#message822884

So, there might be a temporary bug from their side ?

@holtalanm
Copy link
Author

while their api has a free tier, users still need to have an account, and have their domain whitelisted within that account. This is explained in their leaflet documentation, as well:

everything should work seamlessly as long as you have an active plan for your domain

Not sure where that should be communicated within the leaflet-providers codebase, though.

@brunob
Copy link
Member

brunob commented Feb 3, 2020

@holtalanm so you should add this mention in the readme, and we should from our part don't list these tiles in the demo page.

@lseelenbinder
Copy link

Hi @holtalanm, @brunob, we (I'm co-founder of @stadiamaps) just added a property on our side allowing the preview domain (leaflet-extras.github.io). Are there any other domains we should add so there's not a weird grey space where our maps should be? 😄

@jieter
Copy link
Contributor

jieter commented Feb 21, 2020

@lseelenbinder leaflet-extras.github.io should be enough, 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.

4 participants