Skip to content
New issue

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

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

Already on GitHub? # to your account

Add a free vector tile services to avoid the usage of a token #5079

Merged
merged 86 commits into from
Oct 30, 2020

Conversation

Josmorsot
Copy link
Collaborator

@Josmorsot Josmorsot commented Oct 28, 2020

This PR adds CARTO free vector tile services to avoid the usage of a mapboxApiAccessToken in all the examples.

  • Update all the examples: remove mapboxApiAccessToken and add CARTO basemap
  • Add CARTO basemap service at the doc https://deck.gl/docs/get-started/using-with-map
  • Update website
  • Create a new style to match website examples with the current version.
  • Remove mapboxApiAccessToken token from tests

@Josmorsot Josmorsot requested a review from alasarr October 28, 2020 18:02
@Josmorsot Josmorsot marked this pull request as ready for review October 28, 2020 18:04
@Josmorsot Josmorsot changed the title Ivan/carto basemaps Changing basemaps for examples Oct 28, 2020
@Josmorsot Josmorsot changed the title Changing basemaps for examples Change basemaps for examples Oct 28, 2020
website/src/examples/carto.js Outdated Show resolved Hide resolved
website/src/constants/defaults.js Outdated Show resolved Hide resolved
@Pessimistress
Copy link
Collaborator

As discussed, the example READMEs should add a section "Map style" and link to https://deck.gl/docs/get-started/using-with-map

Copy link
Contributor

@alasarr alasarr left a comment

Choose a reason for hiding this comment

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

The most important, add a description to the PR

examples/gallery/src/mapbox-layer.html Show resolved Hide resolved
examples/gallery/src/mapbox-layer.html Outdated Show resolved Hide resolved
examples/website/scenegraph/app.js Outdated Show resolved Hide resolved
modules/carto/src/basemap.js Outdated Show resolved Hide resolved
@alasarr
Copy link
Contributor

alasarr commented Oct 28, 2020

As discussed, the example READMEs should add a section "Map style" and link to https://deck.gl/docs/get-started/using-with-map

Yes, I agreed with @Josmorsot I'll add this part.

@Josmorsot
Copy link
Collaborator Author

The most important, add a description to the PR

Done 👍 Apologies for that.

@Josmorsot Josmorsot changed the title Change basemaps for examples Add a free vector tile services to avoid the usage of a token Oct 28, 2020
@Josmorsot Josmorsot requested a review from alasarr October 30, 2020 11:21
Copy link
Contributor

@alasarr alasarr left a comment

Choose a reason for hiding this comment

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

Cartography is almost there, but I see some differences with the building:

image

Could you check?

docs/api-reference/carto/basemap.md Outdated Show resolved Hide resolved
examples/website/terrain/webpack.config.js Outdated Show resolved Hide resolved
@alasarr
Copy link
Contributor

alasarr commented Oct 30, 2020

If possible, try to reduce the number of commits.

@Josmorsot Josmorsot requested a review from alasarr October 30, 2020 12:39
Copy link
Contributor

@alasarr alasarr left a comment

Choose a reason for hiding this comment

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

Check the boundaries of the first admin region. It looks roads appears before than prod.

image

image

@Josmorsot Josmorsot requested a review from alasarr October 30, 2020 16:17
Copy link
Contributor

@alasarr alasarr left a comment

Choose a reason for hiding this comment

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

All the issues related to cartography are passed.

I've successfully tested (with gatsby):

  • All the website examples (layers, integrations, extensions)
  • Experimentals
  • Playground
  • Layer browser

@Pessimistress Pessimistress merged commit 67c347a into master Oct 30, 2020
@Pessimistress Pessimistress deleted the ivan/carto-basemaps branch October 30, 2020 23:57
Pessimistress pushed a commit that referenced this pull request Oct 31, 2020
Co-authored-by: Alberto Asuero <albertoasuero@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants