Skip to content

Use bounding box from layer in WMTS capabilities#16936

Merged
ahocevar merged 1 commit into
openlayers:mainfrom
remiHau:feat-wmts-bounding-box-capabilities
Jul 2, 2025
Merged

Use bounding box from layer in WMTS capabilities#16936
ahocevar merged 1 commit into
openlayers:mainfrom
remiHau:feat-wmts-bounding-box-capabilities

Conversation

@remiHau

@remiHau remiHau commented Jun 24, 2025

Copy link
Copy Markdown

In the previous version only WGS84 BoundingBox are used to create the tile matrix set from capabilities.
The idea of this feature is to add the possibility to manage projection bounding box from the layer

This is linked to issue #16825

@remiHau remiHau force-pushed the feat-wmts-bounding-box-capabilities branch from 12ebdc8 to ac88cb5 Compare June 24, 2025 14:05
@github-actions

Copy link
Copy Markdown

📦 Preview the website for this branch here: https://deploy-preview-16936--ol-site.netlify.app/.

@ahocevar ahocevar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please create a new test instead of modifying an existing one. This includes the test data.

In the previous version only WGS84 BoundingBox are used to create the tile matrix set from capabilities
The idea of this feature is to add the possibility to manage projection bounding box from the layer
@remiHau remiHau force-pushed the feat-wmts-bounding-box-capabilities branch from ac88cb5 to e683b74 Compare July 1, 2025 15:05
@remiHau

remiHau commented Jul 1, 2025

Copy link
Copy Markdown
Author

I created a new capabilities file for the test and a new describe and I removed the renaming of the existing test even though it might be more confusing to avoid modifying an existing one.

@ahocevar ahocevar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, @remiHau! The reason for creating separate, new tests is to make sure we don't introduce regressions.

@ahocevar ahocevar merged commit 7ce01d2 into openlayers:main Jul 2, 2025
8 checks passed
@remiHau remiHau deleted the feat-wmts-bounding-box-capabilities branch July 2, 2025 08:58
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