Log message #536629

# At Username Text
# Feb 19th 2009, 13:44 AD7six the act of making as current marks the 'current' revision as previous
# Feb 19th 2009, 13:44 AD7six the book revision logic is user submits a pending revision (draft), admin marks it as current or rejected.
# Feb 19th 2009, 13:44 alkemann AD7six: how does the book do drafts (ie keep old version online)
# Feb 19th 2009, 13:43 jperras AD7six: I don't think we will have much use for deleting ratings
# Feb 19th 2009, 13:43 alkemann we may decide against using revision, but we agreed that it was better to create a drafted behavior, than use revision for something it was not intended
# Feb 19th 2009, 13:43 AD7six maybe not needed, it the bakery got spammed it'd be handy to not delete the info you can use to investigate with
# Feb 19th 2009, 13:42 AD7six jperras: to be able to remove without deleting a rating.
# Feb 19th 2009, 13:42 ADmad yeah dont think we are gonna approve ratings too
# Feb 19th 2009, 13:42 jperras what is the 'status' field in the rating supposed to be
# Feb 19th 2009, 13:42 ADmad heh
# Feb 19th 2009, 13:41 AD7six ADmad: yes :)
# Feb 19th 2009, 13:41 alkemann AD7six: revision and drafts are very different creatures
# Feb 19th 2009, 13:41 alkemann it's not in use cause the code is barely baked.
# Feb 19th 2009, 13:41 ADmad AD7six: isnt the ratings table missing a filed for storing the rating ?
# Feb 19th 2009, 13:40 AD7six why do you need both revision and drafted behaviors
# Feb 19th 2009, 13:39 AD7six I don't see it used anywhere - I assume it's intended to be used in the page model
# Feb 19th 2009, 13:36 alkemann yes, it was made for bakery
# Feb 19th 2009, 13:36 AD7six I am unfamiliar with the drafted behavior :)
# Feb 19th 2009, 13:36 alkemann AD7six: i am unfamiliar with the book code.
# Feb 19th 2009, 13:35 alkemann http://thechaw.com/bakery/source/branches/2.0.x.x/models/behaviors/drafted.php
# Feb 19th 2009, 13:35 AD7six alkemann: imo it would be beneficial to use the same logic as the book regarding pending submissions - because there's then only 1 code base to edit in that regard
# Feb 19th 2009, 13:35 alkemann AD7six: i guess it slipped by .sorry
# Feb 19th 2009, 13:33 AD7six alkemann: where is the drafted behavior mentioned in the spec (link)
# Feb 19th 2009, 13:33 AD7six updated: http://img518.imageshack.us/img518/8728/schematicmneatods6.png
# Feb 19th 2009, 13:32 alkemann agreed
# Feb 19th 2009, 13:32 jperras a simple identity map should be good enough for comment types. no need for an extra table
# Feb 19th 2009, 13:32 alkemann AD7six: no. draftedbehavior.. leaving articles online, while edits are pending moderation
# Feb 19th 2009, 13:31 AD7six the spec says comment types are fixed in the system I've linked to an old image that table shouldn't exist
# Feb 19th 2009, 13:31 ADmad since it doesnt do much other than specify in which tab the comment is shown
# Feb 19th 2009, 13:30 ADmad do we need a comment_types table? can't just do with a string field in comments
# Feb 19th 2009, 13:29 AD7six changed
# Feb 19th 2009, 13:28 ADmad AD7six: for comments table created modified should be datetime
# Feb 19th 2009, 13:26 AD7six alkemann: are you referring to your publish by voting discussion?
# Feb 19th 2009, 13:25 ADmad *i dont
# Feb 19th 2009, 13:25 ADmad i want to have to click each time i want to read a comment
# Feb 19th 2009, 13:25 ADmad "only comment titles are visible, must expand to see comment body" not sure i like that..
# Feb 19th 2009, 13:24 AD7six http://img15.imageshack.us/img15/6152/schematicmneatonf4.png next?
# Feb 19th 2009, 13:20 AD7six cool
# Feb 19th 2009, 13:20 alkemann just a matter of what tab they are in i think
# Feb 19th 2009, 13:19 AD7six is there planned to be any logic in the app based on comment types
# Feb 19th 2009, 13:19 alkemann last as in regards to it's siblings