ckanext-spatial icon indicating copy to clipboard operation
ckanext-spatial copied to clipboard

Adding validator and parser for default_groups and tags config

Open ebuckley opened this issue 6 years ago • 2 comments

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 avatar Apr 20 '18 03:04 ebuckley

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

ebuckley avatar Apr 22 '18 21:04 ebuckley

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

amercader avatar Apr 27 '18 13:04 amercader