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

support multiple datasets, uploading non-tabulr resources, dump log #10

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

OriHoch
Copy link

@OriHoch OriHoch commented Sep 16, 2018

This PR contains several changes which I needed for ckanext-upload_via_email:

  • Support loading multiple datasets, providing the dataset properties dynamically.
  • Support for uploading non-tabular resources to CKAN (not just storing the URL)
  • Output a resource with log of created datasets/resources or error details

Also some minor changes:

  • Support setting the ckan host via env var
  • Support for setting resource name with special chars
  • Allow to extend the add_ckan_resource processor

I tried to keep backwards compatibility so existing pipelines shouldn't be affected by the changes

See updated docs and test for usage details.

@roll roll added the review label Apr 15, 2019
@roll
Copy link
Member

roll commented Apr 15, 2019

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

@roll roll removed the review label Jun 3, 2019
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.

2 participants