[BUG] Fix "STAC viewer" links pointing to raw catalog JSON - #481
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Signed-off-by: John McCall <john@overturemaps.org>
Super-linter summary
All files and directories linted successfully For more information, see the GitHub Actions workflow run Powered by Super-linter |
🗺️ OMF Docs previews are live!
Auto-gen schema site is now available. This is an early preview of a future workflow where we will automatically generate and publish reference docs for the Overture Maps Format schema. The auto-gen schema site may contain incomplete or inaccurate information as we are still refining the generation process, so please compare against the repo schema site and refer to the official Overture documentation for authoritative information. Note ♻️ This preview updates automatically with each push to this PR. |
There was a problem hiding this comment.
Pull request overview
Updates the documentation to fix misleading “STAC viewer” links that currently land users on raw catalog JSON, and removes a broken MapLibre demo that relied on stale tile URLs.
Changes:
- Repoints “STAC viewer” doc links to a hosted stac-browser instance for browsing
stac.overturemaps.org/catalog.json. - Removes the MapLibre/PMTiles embedded demo and associated style JSON tabs from the “build-a-map” example page.
- Deletes the now-unused React MapLibre demo implementation (
map.js) and its style definitions (layers.js).
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
src/components/map.js |
Deletes the embedded MapLibre demo component that referenced stale PMTiles URLs. |
src/components/layers.js |
Deletes the MapLibre style-layer definitions used only by the removed demo. |
docs/getting-data/quick-start.mdx |
Updates the “STAC viewer” link to point to stac-browser. |
docs/getting-data/cloud-sources.mdx |
Updates both “STAC viewer” references to point to stac-browser. |
docs/examples/build-a-map.mdx |
Removes the embedded demo map and the style tabs; updates surrounding copy accordingly. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Dana Bauer (danabauer)
left a comment
There was a problem hiding this comment.
Thanks for doing this, John!
https://stac.overturemaps.org/returns the rawCatalogJSON, not a browsable UI. Fixes #460.The three "STAC viewer" links (
cloud-sources.mdxx2,quick-start.mdx) now point at a hosted stac-browser instance (https://browser.moregeo.it/external/stac.overturemaps.org/catalog.json) instead.Also dropped the MapLibre demo on the
build-a-mappage (src/components/map.js+layers.js). It fetched PMTiles from a stale CloudFront URL (d3c1b7bog2u1nn.cloudfront.net/examples/*.pmtiles) that no longer serves those tiles, so the embed was broken. Rewiring it totiles.overturemaps.orgwould also require rewritinglayers.js's style rules against the real tile schema (source-layer,@namevsname,divisionsubtypes replacing the oldlocality_type/localityType/subclassfields that don't exist in current tiles), which is more surface area than this page needs.The
buildings-map.jscomponent on other pages is unaffected: it already builds its style rules againsttiles.overturemaps.organd the current schema.