Skip to content

Conversation

@m-mohr
Copy link
Collaborator

@m-mohr m-mohr commented Sep 16, 2025

See stac-extensions/web-map-links#28

Potential issue:

  • OpenLayers can't create a tile matrix set from the URL template and fails => solution: Provide the capabilities document in the href and the URL template (optionally) in the uriTemplate.
  • OpenLayers loads the tiles via HTML <img>-tag, which means we can't send any headers to the server

@m-mohr m-mohr requested a review from Copilot September 16, 2025 15:30
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements WMTS REST request encoding support for STAC layers. The change adds the ability to handle WMTS links that use REST-style URLs with template variables instead of the traditional KVP (Key-Value Pair) encoding.

  • Adds support for WMTS REST request encoding by parsing template variables from link metadata
  • Implements variable substitution logic using const, default, or first enum values
  • Restructures existing WMTS handling to support both REST and KVP encoding methods

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/ol/layer/STAC.js Adds REST encoding support with variable substitution and restructures WMTS case handling
CHANGELOG.md Documents the new WMTS REST request encoding feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@m-mohr m-mohr added this to the 1.1.0 milestone Nov 24, 2025
@m-mohr m-mohr merged commit f750dff into main Nov 26, 2025
3 checks passed
@m-mohr m-mohr deleted the wmts-rest branch November 26, 2025 17:03
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