-
Notifications
You must be signed in to change notification settings - Fork 685
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
adding geoportail france provider (new try) #284
adding geoportail france provider (new try) #284
Conversation
Hi and thanks for the patch, two points :
|
Hi and thanks for paying attention to this PR. Corrections done. |
"De rien" & thx for the fixes, GTM @jieter ? |
@gcebelieu @brunob is is a good idea to add this API key? I have an api key for HERE here, it gets a lot of views, regularly over 10M/month. That is without being in the default distribution, only views from the preview. |
@gcebelieu can tell us more about it since he seems to work for IGN which is linked to the provider ;) |
@brunob, indeed :) |
👍 |
This PR adds Geoportail France provider to leaflet-providers. Same goal as @fionnh previous unmerged PR (#279). The code comes from Mediawiki Extension:Maps latest evolution (ProfessionalWiki/Maps#437) which uses leaflet-providers as a dependency.