Ticket #2581 (closed enhancement: fixed)
allow state = 'draft'
Reported by: | toby | Owned by: | toby |
---|---|---|---|
Priority: | awaiting triage | Milestone: | demo phase 2 |
Component: | ckan | Keywords: | demo-theme |
Cc: | Repository: | ckan | |
Theme: | none |
Description
pass some stuff through the context
Change History
comment:2 Changed 23 months ago by toby
- Owner changed from toby to kindly
- Status changed from new to assigned
fixed in commit 0f84d99 in the demo branch
david can you just look at this for a sanity check - I'm just concerned that only state benefits from this
if good reassign to me and i'll remove the fixme
comment:3 Changed 22 months ago by kindly
This looks fine, just make doubly sure that if this flag is set then whatever sets it explicitly sets the state i.e overrides what the user sent.
comment:5 Changed 22 months ago by toby
- Owner changed from toby to kindly
@kindly
we actually need an extra state for full functionality can we create a new 'draft-complete' state as well? does that need any additional magic?
please answer and reassign
comment:6 Changed 22 months ago by kindly
- Owner changed from kindly to toby
No any state without active, pending or deleted in their name is fine.
do before
data.pop(key)