Ticket #2558 (closed enhancement: fixed)

Opened 23 months ago

Last modified 23 months ago

Demo site add dataset has missing functionality

Reported by: aron.carroll Owned by: aron.carroll
Priority: blocker Milestone: demo phase 1
Component: ckan Keywords: demo-theme
Cc: toby Repository: ckan
Theme: none

Description

  • Needs tags
  • Needs groups
  • Needs slug
  • Licenses links to something
  • Needs tooltips (Ira to write some tooltips)

Change History

comment:1 Changed 23 months ago by aron.carroll

  • Milestone changed from current-ckan-sprint-2012-06-25 to demo phase 1

comment:2 Changed 23 months ago by aron.carroll

  • Owner changed from aron.carroll to toby
  • Status changed from new to assigned

Started this as of 4d3c87a however tags need to be hooked up (apparently they can only be added on update at the moment). Also by moving the groups to the metadata form I now get integrity errors while saving. Toby could you please take a look at this.

comment:3 Changed 23 months ago by aron.carroll

  • Priority changed from major to blocker

comment:4 Changed 23 months ago by toby

@aron,

not quite sure what your problem was. I had an integrity error on part 1 save which is fixed.

my issues

1) part 1 tags not saved

2) part 2 resources not saved

3) part 3 no form shown - i thought you had one showing where did it go?

will look at 1 & 2 for starters

comment:5 Changed 23 months ago by toby

1) tags save but don't show in dataset edit - will push fix soon

comment:6 Changed 23 months ago by toby

  • Owner changed from toby to aron.carroll

2) resources save but not sure if being validated etc needs testing but working to an extent

@aron,

I'm pushing this back to you now reassign as needed

comment:7 Changed 23 months ago by aron.carroll

  • Owner changed from aron.carroll to toby

Metadata is back as of b20538f

Groups currently aren't showing because the {{{c.groups_available}} param isn't available.

https://github.com/okfn/ckan/blob/feature-2375-demo-theme/ckan/templates/package/new_package_metadata.html#L7

comment:8 Changed 23 months ago by toby

groups available now is a helper function (templates updated) commit d0506f3

comment:9 Changed 23 months ago by toby

@aron,

metadata now saves pushing back to you - in branch

comment:10 Changed 23 months ago by toby

  • Owner changed from toby to aron.carroll

comment:11 Changed 23 months ago by aron.carroll

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

Marking as fixed. Will create new issues as they arise

Note: See TracTickets for help on using tickets.