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

Add maplibre in docs intro #2384

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

birkskyum
Copy link

Added MapLibre GL JS to the docs description/landing next to Mapbox GL JS.

@@ -4,7 +4,7 @@
Depending on which base map library you use, you may need a Mapbox token. You will need a Mapbox token if you use:

- [mapbox-gl@>=2.0.0](https://github.com/mapbox/mapbox-gl-js/releases/tag/v2.0.0) - requires a mapbox access token in order to access the map renderer, and generates billable events regardlesss of whether you are displaying your own maps.
- `mapbox-gl@1.x` or `maplibre-gl@1.x` - requires an access token only if you load the map styles and tiles from Mapbox's data service. See "Display Maps Without A Mapbox Token" section below for using non-Mapbox tiles.
- `mapbox-gl@1.x` - requires an access token only if you load the map styles and tiles from Mapbox's data service. See "Display Maps Without A Mapbox Token" section below for using non-Mapbox tiles.
Copy link
Author

@birkskyum birkskyum May 2, 2024

Choose a reason for hiding this comment

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

Removing maplibre here, because if the style comes from the Mapbox's API, then a mapbox renderer should be used to render it in order to stay compliant with mapbox license.

https://github.com/mapbox/mapbox-gl-js/blob/main/src/style-spec/LICENSE.txt

The software and files in this repository (collectively, "Software") are
licensed under the Mapbox TOS for use only with the relevant Mapbox product(s)

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

1 participant