Ticket #1608 (closed enhancement: fixed)
Merge storage into Core
Reported by: | ross | Owned by: | ross |
---|---|---|---|
Priority: | awaiting triage | Milestone: | ckan-sprint-2012-01-23 |
Component: | ckan | Keywords: | storage 5d |
Cc: | kindly | Repository: | ckan |
Theme: | none |
Description (last modified by rgrp) (diff)
As part of Storage Changes and http://ckan.okfnpad.org/newstorage it was decided to merge ckanext-storage back into core.
Other changes:
- Remove /storage/upload page (we have frontend integration elsewhere now and we should also ensure that upload items relate to a resource or other domain object).
- Provide a local path as well as remote path for form upload requests in API so that frontend can see url to CKAN path rather than cloud storage path ...
Change History
comment:2 Changed 2 years ago by ross
The current storage API might need cleaning up and merging into the normal API. Although this may require changes to the current API Controller. Not sure, so adding a note as a reminder.
comment:6 Changed 2 years ago by rgrp
- Status changed from closed to reopened
- Resolution fixed deleted
- Description modified (diff)
Want to re-enable direct upload to cloud storage.
comment:8 Changed 2 years ago by rgrp
- Status changed from reopened to closed
- Resolution set to fixed
Fixed in https://github.com/okfn/ckan/commit/4488795a54866a02305268d77a2648fa85538ee8
And working on <http://thedatahub.org/>, test etc.
One important note: test using remote storage has been disabled -- see https://github.com/okfn/ckan/commit/3c9ddd31d741eb8ec8c145063ce967e781947a88
Time spent: 0.25d.
We should consider some admin dashboard hooks for data such as total file storage usage+count