search

frictionlessdata / datapackage-pipelines-ckan

Data Package Pipelines processors for CKAN
MIT License
1 stars 1 forks source link

allow to specify stream headers arg, support translated error message #11

Open OriHoch opened 6 years ago

OriHoch commented 6 years ago

fixes 2 issues in dump.to_ckan processor:

  1. If default locale is not english, error message for existing resource is translated which prevents overwrite_existing parameter from working. Fixed by adding an optional parameter resource-exists-error-message which should contain the translated resource exists error message for default locale of the target CKAN instance.
  2. Headers in source file appear in first row of data in the datastore, fixed by adding an optional parameter stream-resource-headers which allows to specify the number of headers to skip.
roll commented 5 years ago

@akariv @OriHoch We need someone else (not Brook) to review it..