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

feat(ios) (issue #7664) Add PMTiles to mediaExtensions #7663

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

plasticviking
Copy link

PMTiles relies on byte range requests to serve vector tiles. See https://github.com/protomaps/PMTiles/blob/6f54794/js/index.ts#L380 for example of client request.

PMTiles relies on byte range requests to serve vector tiles.
@jcesarmobile
Copy link
Member

Can you report an issue first?

@jcesarmobile jcesarmobile added the needs reply needs reply from the user label Sep 18, 2024
@plasticviking plasticviking changed the title Add PMTiles to mediaExtensions feat( #7664 Add PMTiles to mediaExtensions Sep 18, 2024
@plasticviking plasticviking changed the title feat( #7664 Add PMTiles to mediaExtensions feat(ios) (issue #7664) Add PMTiles to mediaExtensions Sep 18, 2024
@plasticviking
Copy link
Author

@jcesarmobile Issue #7664 created to describe the proposed change

@Ionitron Ionitron removed the needs reply needs reply from the user label Sep 18, 2024
@m345054
Copy link

m345054 commented Dec 2, 2024

Will this get merged soon?

@jcesarmobile
Copy link
Member

No, the linked issue was closed because the reporter didn't provide a valid reproduction.
If you are facing this problem, please, report a new issue and provide a sample app that reproduces it.

@m345054
Copy link

m345054 commented Dec 2, 2024

Well it's a feature request not a problem. It's useful for offline maps with maplibre or leaflet.
I could provide some sample code to render the map and load from a pmtiles file if it's required.

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.

4 participants