# |
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 |
# |
Feb 19th 2009, 13:11 |
ADmad |
this is a tiny but important feature... dont want ppl to be refering to stale stuff like those printed books |
# |
Feb 19th 2009, 13:10 |
AD7six |
good point |
# |
Feb 19th 2009, 13:10 |
alkemann |
this tag is not jsut a visual thing on the article page, it removes the article from several other features. like search and front page |
# |
Feb 19th 2009, 13:09 |
alkemann |
nope |
# |
Feb 19th 2009, 13:09 |
AD7six |
alkemann: to put it another way is there any other kind of big-red-box-or-other effect you're planning on adding |
# |
Feb 19th 2009, 13:09 |
alkemann |
they do now. dont see why we should stop that |
# |
Feb 19th 2009, 13:09 |
jperras |
because a normal user shouldn't be able to tag an article as deprecated |
# |
Feb 19th 2009, 13:08 |
AD7six |
oh I see because users are going to be able to tag things themselves? |
# |
Feb 19th 2009, 13:08 |
jperras |
and then need to blacklist tags (or use a pre-defined dropdown of tags) for user input |
# |
Feb 19th 2009, 13:08 |
alkemann |
it's just infinitly easier and also more effective to have a boolean in article table |
# |
Feb 19th 2009, 13:08 |
AD7six |
how so, or is the concept of tags changing for the bakery |