| # |
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 |
| # |
Feb 19th 2009, 13:19 |
AD7six |
ok. |
| # |
Feb 19th 2009, 13:19 |
alkemann |
no matter who u reply to, your reply is posted last with the original comment as parent |
| # |
Feb 19th 2009, 13:18 |
AD7six |
and the 1 level deep bit? |
| # |
Feb 19th 2009, 13:15 |
alkemann |
if u post a new comment it is a root node. if u reply it is a level 2 node |
| # |
Feb 19th 2009, 13:15 |
AD7six |
what does "Threaded replies, one level deep" mean |
| # |
Feb 19th 2009, 13:14 |
AD7six |
k |
| # |
Feb 19th 2009, 13:14 |
alkemann |
AD7six: please read the specc stuff. we have changed the edit moderation system |
| # |
Feb 19th 2009, 13:13 |
AD7six |
ADmad: pending edit |
| # |
Feb 19th 2009, 13:13 |
ADmad |
AD7six: do we need a status for page too ? its not as if one page on article will be offline and others not |
| # |
Feb 19th 2009, 13:12 |
alkemann |
the comment stuff doesnt look like it matches specc |
| # |
Feb 19th 2009, 13:11 |
AD7six |
so whatelse http://img26.imageshack.us/img26/1732/schematicmneatooc5.png |
| # |
Feb 19th 2009, 13:11 |
alkemann |
example : http://debuggable.com/posts/google-analytics-php-api-cakephp-model:480f4dd5-b2a4-452a-b4ed-4f3fcbdd56cb |
| # |
Feb 19th 2009, 13:11 |
jperras |
add in the complexity that I described with permissions, and it's not a very scalable implementation |
