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 support for Mapbox Vector Tile to the tile server #678

Closed
wants to merge 1 commit into from

Conversation

broot5
Copy link
Contributor

@broot5 broot5 commented Jan 13, 2025

Mapbox Vector Tile is a file format for delivering geographic data as vector-based tiles.
Previously, tiled GeoJSON had limited use cases. With Mapbox Vector Tile, the Plus Codes Grid Service can now be utilized by a wider variety of clients.

Copy link

google-cla bot commented Jan 13, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@broot5 broot5 force-pushed the mvt branch 2 times, most recently from d992b5d to a199574 Compare January 13, 2025 23:09
@drinckes
Copy link
Contributor

Hi @broot5 , do you have a running instance of this for a preview? If so can you post a URL to it?

@drinckes
Copy link
Contributor

Also, it looks like this is doing at least two things - updating the GeoJSON packages and adding MVT support. In the interests of keeping changes focused and small, could you split it into two please?

Replace deprecated package io/ioutil with os
@broot5
Copy link
Contributor Author

broot5 commented Jan 14, 2025

Hi @broot5 , do you have a running instance of this for a preview? If so can you post a URL to it?

I do not have a running instance now, but I will try to set one up if needed.

Also, it looks like this is doing at least two things - updating the GeoJSON packages and adding MVT support. In the interests of keeping changes focused and small, could you split it into two please?

Got it. I will close this PR and open separate PRs.

@broot5 broot5 closed this Jan 14, 2025
@broot5 broot5 deleted the mvt branch January 14, 2025 14:42
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.

2 participants