Skip to content

Comments

Removing danglish js/html files that are not actually used to do anything#1477

Merged
aaime merged 1 commit intoGeoWebCache:mainfrom
aaime:dangling_js
Feb 20, 2026
Merged

Removing danglish js/html files that are not actually used to do anything#1477
aaime merged 1 commit intoGeoWebCache:mainfrom
aaime:dangling_js

Conversation

@aaime
Copy link
Member

@aaime aaime commented Feb 19, 2026

Probabably a work in progress that never got completed, from 12 years ago

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 removes unused OpenLayers 3 web resource files that appear to be incomplete work from 12 years ago. The files being deleted include WMTS and WMS JavaScript implementations along with their HTML pages, plus a minified jQuery library.

Changes:

  • Deletion of unused OpenLayers 3 WMTS implementation files
  • Deletion of unused OpenLayers 3 WMS implementation files
  • Removal of bundled jQuery 1.11.1 library

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
geowebcache/core/src/main/resources/org/geowebcache/rest/webresources/openlayers3/wmts.js Removes incomplete WMTS JavaScript implementation for OpenLayers 3
geowebcache/core/src/main/resources/org/geowebcache/rest/webresources/openlayers3/wmts.html Removes HTML page for WMTS OpenLayers 3 demo
geowebcache/core/src/main/resources/org/geowebcache/rest/webresources/openlayers3/wms.js Removes incomplete WMS JavaScript implementation for OpenLayers 3
geowebcache/core/src/main/resources/org/geowebcache/rest/webresources/openlayers3/wms.html Removes HTML page for WMS OpenLayers 3 demo
geowebcache/core/src/main/resources/org/geowebcache/rest/webresources/openlayers3/jquery-1.11.1.min.js Removes bundled jQuery 1.11.1 minified library

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@dromagnoli dromagnoli left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@aaime aaime merged commit bd844f4 into GeoWebCache:main Feb 20, 2026
18 checks passed
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