-
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
Introduce HEREv3 (using apiKey instead of app_id and app_code) #343
Conversation
- Tile API URL changed - Authorization by apiKey
@jieter any thoughts ? |
Yes, the readme links to the preview, using leaflet-providers/preview/preview.js Line 81 in 52bfb50
The reason being that we kept exceeding the free plan, even with a single layer in the preview list. I'd say there is limited value in adding these providers we cannot reliably show previews for, as they are not really discoverable. |
LGTM |
Merged, thanks @mcguffin |
Can someone tag 1.9.2 with this update? Thanks! |
@stell no hurry, we were waiting to merge others PR, now that it's done, we will shortly publish a new tag when our free time will allow it :) |
Use Tile API-URL and apiKey as described in latest HERE docs: https://developer.here.com/documentation/map-tile/dev_guide/topics/quick-start-map-tile.html
Will fix #342