Ticket #2256 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

remove TDH from core

Reported by: toby Owned by: toby
Priority: awaiting triage Milestone:
Component: ckan Keywords:
Cc: Repository: ckan
Theme: none


make core independent of TDH and make core as minimal app

Change History

comment:1 Changed 2 years ago by toby

I'd like to remove all 'custom' icons and just use bootstrap ones - is this reasonable? they are added in different ways but h.icon() could be easily adapted for this.

minimal .sj .css files

comment:2 Changed 2 years ago by rgrp

Not sure what ticket means (remove TDH from core?). Also agree that we should glyphicons from bootstrap where possible going forward - though that would seem to fit more naturally with #2224 (Simplify javascript and css dependencies and add minified version) and have little to do with TDH versus core :-)

comment:3 Changed 2 years ago by toby


The idea of this ticket is to separate TDH into an extension rather than it being incorporated into core. This should help TDH by making it possible to push fixes without needing to do a new release of core. It should also help make core more stable and as slim line as possible. New features can be tested in TDH and then if they have a wider appeal they can be merged into core.

There is a strong desire for this within the dev team.

comment:4 Changed 2 years ago by rgrp

@toby: indeed and I'm strongly in favour of the basic idea too (and largely a dev question so up to you guys though feature / functionality implications would need to be talked through). My point was that custom icon stuff was #2224 :-)

comment:5 Changed 2 years ago by toby


Thanks, I think there is overlap and I'm happy to leave that bit of work to you (or future ticket owner) I was just adding some brief notes to this ticket on some of the outcomes I'd like to see and hence why I'd be happy to simplify the icons.

comment:6 Changed 2 years ago by toby

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

I'm closing this ticket now ckanext-datahub exists and is live - the other work is included in other tickets

Note: See TracTickets for help on using tickets.