ckan / ckanext-spatial

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

Adding validator and parser for default_groups and tags config #196

Open ebuckley opened 6 years ago

ebuckley commented 6 years ago

Context

Ckan NZ is running a fork of ckanext-spatial which includes a few extra validation steps and a few modifications that make source config closer to the base ckan harvester behaviour.

Changes

  1. add the same validation as base CKANHarvester for the source config object
  2. adds support for default groups in the spatial harvester
  3. Modifies expected harvest source configuration for tags to match the behavior of CKANHarvester
ebuckley commented 6 years ago

Happy to update tests to support this change if there is some desire to have it contributed back in to core :)

amercader commented 6 years ago

@ebuckley this looks great, it would be great if you could update the tests (and test this new behaviour) thanks