ckan / ckanext-spatial

Geospatial extension for CKAN
http://docs.ckan.org/projects/ckanext-spatial
126 stars 193 forks source link

[#162] Allow use of WMS layer in extent map #163

Closed etj closed 8 years ago

davidread commented 8 years ago

Code looks fine. Apologies if I'm being ignorant, but why would someone choose to put a base map in a WMS layer?

etj commented 8 years ago

Good question :) I was porting an old CKAN instance to a later version, and the original setup used WMS, so I simply needed to replicate that environment.

Anyway, I created the PR bc it simply offers one more option to the CKAN admin.

davidread commented 8 years ago

@etj thanks for that.

@amercader it's up to you as maintainer

amercader commented 8 years ago

I'd never advocate this use of WMS but if it's helpful for migrating legacy apps I'm happy to merge it the code is quite straight-forward. Thanks @etj