Ticket #1608 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

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:1 Changed 2 years ago by ross

We should consider some admin dashboard hooks for data such as total file storage usage+count

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:3 Changed 2 years ago by kindly

  • Keywords 5d added

comment:4 Changed 2 years ago by kindly

  • Milestone set to current-ckan-sprint-2012-01-23

comment:5 Changed 2 years ago by ross

  • Status changed from new to closed
  • Resolution set to fixed

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:7 Changed 2 years ago by rgrp

  • Description modified (diff)

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.

comment:9 Changed 2 years ago by dread

This has gone into CKAN 1.6

Note: See TracTickets for help on using tickets.