# |
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 |