[BUG](overture-tiles) use QueryBuilder component for dynamic release paths for tile URL - #469
Merged
John McCall (lowlydba) merged 2 commits intoAug 10, 2026
Conversation
TAKAHASHI Shuuji (shuuji3)
changed the title
TAKAHASHI Shuuji (shuuji3)
force-pushed
the
fix/pmtiles-use-latest-release-date
branch
from
August 8, 2026 07:15
1ae70ce to
83f3ed7
Compare
…aths for tile URL Signed-off-by: TAKAHASHI Shuuji <id@shuuji3.xyz>
TAKAHASHI Shuuji (shuuji3)
force-pushed
the
fix/pmtiles-use-latest-release-date
branch
from
August 8, 2026 07:25
83f3ed7 to
2a90977
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the PMTiles “Overture tiles” documentation to stop hardcoding release dates by using the existing QueryBuilder component, and extends QueryBuilder to support inline code and link rendering with placeholder substitution.
Changes:
- Added placeholder substitution helper and new rendering modes (
href→<a>,inline→<code>) toQueryBuilder. - Replaced hardcoded release strings in
docs/examples/overture-tiles.mdxwithQueryBuilderplaceholders so URLs/snippets track the latest configured release.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/components/queryBuilder.js | Refactors placeholder replacement and adds link/inline rendering modes to support dynamic release strings in more contexts. |
| docs/examples/overture-tiles.mdx | Replaces hardcoded release date strings with QueryBuilder placeholders for PMTiles URLs and command examples. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: TAKAHASHI Shuuji <id@shuuji3.xyz>
John McCall (lowlydba)
approved these changes
Aug 10, 2026
John McCall (lowlydba)
left a comment
John McCall (lowlydba)
left a comment
Contributor
There was a problem hiding this comment.
TAKAHASHI Shuuji (@shuuji3) Thanks for the issue and fix! The NATURAL_LANGUAGE lint error is pre-existing, so we can ignore that for this context.
John McCall (lowlydba)
merged commit Aug 10, 2026
c503c4d
into
OvertureMaps:main
13 of 14 checks passed
TAKAHASHI Shuuji (shuuji3)
deleted the
fix/pmtiles-use-latest-release-date
branch
August 10, 2026 15:56
Contributor
Author
|
Confirmed the fix. Thanks! https://docs.overturemaps.org/examples/overture-tiles/ 
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
resolve #470
This PR replaced the hardcoded date with
QueryBuildercomponent to ensure to dynamically generate the latest date.I also update the
QueryBuildercomponent as it doesn't support the inline<code>element. Now when you specity newinlineattribute, it instead returns<code>element.Marked strings are now dynamically updated: