Skip to content

[BUG] Fix "STAC viewer" links pointing to raw catalog JSON - #481

Merged
John McCall (lowlydba) merged 1 commit into
mainfrom
lowlydba-issue-460
Aug 12, 2026
Merged

[BUG] Fix "STAC viewer" links pointing to raw catalog JSON#481
John McCall (lowlydba) merged 1 commit into
mainfrom
lowlydba-issue-460

Conversation

@lowlydba

@lowlydba John McCall (lowlydba) commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

https://stac.overturemaps.org/ returns the raw Catalog JSON, not a browsable UI. Fixes #460.

The three "STAC viewer" links (cloud-sources.mdx x2, 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-map page (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 to tiles.overturemaps.org would also require rewriting layers.js's style rules against the real tile schema (source-layer, @name vs name, division subtypes replacing the old locality_type/localityType/subclass fields that don't exist in current tiles), which is more surface area than this page needs.

The buildings-map.js component on other pages is unaffected: it already builds its style rules against tiles.overturemaps.org and the current schema.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Signed-off-by: John McCall <john@overturemaps.org>
@github-actions

Copy link
Copy Markdown

Super-linter summary

Language Validation result
JAVASCRIPT_ES Pass ✅
MARKDOWN Pass ✅
NATURAL_LANGUAGE Pass ✅
SQLFLUFF Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

@github-actions

Copy link
Copy Markdown

🗺️ OMF Docs previews are live!

🆕 Auto-gen schema site (beta) https://staging.overturemaps.org/docs/pr/481/index.html
🗂️ Auto-gen schema ref main@bd85f08
🌍 Repo schema site https://staging.overturemaps.org/docs/pr/481-no-autogen/index.html
🕐 Updated Aug 12, 2026 21:00 UTC
📝 Commit 89a17f9

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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@danabauer Dana Bauer (danabauer) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for doing this, John!

@lowlydba
John McCall (lowlydba) merged commit fc89512 into main Aug 12, 2026
23 checks passed
@lowlydba
John McCall (lowlydba) deleted the lowlydba-issue-460 branch August 12, 2026 21:04
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.

[BUG] "STAC viewer" links point to raw catalog JSON, not a viewer

3 participants