Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #1217, fixes #1303, fixes #1304 adding WMTS support to Leaflet,… #1397

Merged
merged 2 commits into from
Jan 25, 2017

Conversation

mbarto
Copy link
Contributor

@mbarto mbarto commented Jan 24, 2017

… Openlayers and Catalog

…tions-it#1304 adding WMTS support to Leaflet, Openlayers and Catalog
@simboss
Copy link
Member

simboss commented Jan 24, 2017

At last!

Never loose hope...

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 80.459% when pulling 0e2da7e on mbarto:wmts_layers_support into bd4bc34 on geosolutions-it:master.

@mbarto mbarto requested review from offtherailz and Gnafu January 24, 2017 14:18
<MatrixHeight>32768</MatrixHeight>
</TileMatrix>
<TileMatrix>

Copy link
Member

Choose a reason for hiding this comment

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

generic url instead?

<MaxTileRow>5</MaxTileRow>
<MinTileCol>14</MinTileCol>
<MaxTileCol>14</MaxTileCol>
</TileMatrixLimits>
Copy link
Member

Choose a reason for hiding this comment

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

Can we reduce the number of levels to have a smaller file ?

@@ -0,0 +1,38 @@
/**
Copy link
Member

Choose a reason for hiding this comment

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

Is this file needed anymore ?

offtherailz
offtherailz previously approved these changes Jan 25, 2017
"group": "Meteo",
"format": "image/png",
"tileMatrixSet": "EPSG:900913",
"url": "http://demo.geo-solutions.it/geoserver/gwc/service/wmts"
Copy link
Member

Choose a reason for hiding this comment

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

generic url

"group": "Meteo",
"format": "image/png",
"tileMatrixSet": "EPSG:900913",
"url": ["http://demo.geo-solutions.it/geoserver/gwc/service/wmts", "http://demo.geo-solutions.it/geoserver/gwc/service/wmts"]
Copy link
Member

Choose a reason for hiding this comment

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

generic url for tests

"group": "Meteo",
"format": "image/png",
"tileMatrixSet": "EPSG:900913",
"url": "http://demo.geo-solutions.it/geoserver/gwc/service/wmts"
Copy link
Member

Choose a reason for hiding this comment

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

generic url

"group": "Meteo",
"format": "image/png",
"tileMatrixSet": "EPSG:900913",
"url": ["http://demo.geo-solutions.it/geoserver/gwc/service/wmts", "http://demo.geo-solutions.it/geoserver/gwc/service/wmts"]
Copy link
Member

Choose a reason for hiding this comment

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

generic url for testing

@@ -0,0 +1,1913 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

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

Can we reduce size of this files (e.g. less levels/layers)?

<ows:ServiceTypeVersion>1.0.0</ows:ServiceTypeVersion>
</ows:ServiceIdentification>
<ows:ServiceProvider>
<ows:ProviderName>http://demo.geo-solutions.it/geoserver/gwc/service/wmts</ows:ProviderName>
Copy link
Member

Choose a reason for hiding this comment

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

and we should replace all theese urls?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 80.346% when pulling 22ab4e3 on mbarto:wmts_layers_support into bd4bc34 on geosolutions-it:master.

@mbarto mbarto merged commit 3ab2155 into geosolutions-it:master Jan 25, 2017
@mbarto mbarto deleted the wmts_layers_support branch January 25, 2017 17:36
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.

5 participants