bellroy / lesswrong-migrated

Automatically exported from code.google.com/p/lesswrong
Other
0 stars 0 forks source link

Comment threads can's be banned without trace #277

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Currently, if you ban a thread of comments (say, 3 comments, each replying to 
the previous one other, A, B and C), the result, as seen by a logged-off user, 
is two "Comment deleted" entries (placeholders corresponding to A and B, I 
assume). This doesn't allow banning threads without a trace. (In contrast, when 
a single comment is banned, it disappears completely.)

Original issue reported on code.google.com by robot...@gmail.com on 26 Nov 2011 at 5:41

GoogleCodeExporter commented 9 years ago
I changed this to work the way one would expect, hiding deleted comments unless 
they have visible descendants.  https://github.com/tricycle/lesswrong/pull/12

Original comment by mquan...@gmail.com on 12 Dec 2011 at 12:26

GoogleCodeExporter commented 9 years ago
Was it only for new comments? The following thread still has this problem:
  http://lesswrong.com/r/discussion/lw/8l5/facingthesingularitycom_discussion_page/5bpg

Original comment by robot...@gmail.com on 16 Dec 2011 at 12:03

GoogleCodeExporter commented 9 years ago
(Oh, I guess it's not in production yet.)

Original comment by robot...@gmail.com on 16 Dec 2011 at 12:04

GoogleCodeExporter commented 9 years ago
It was deployed briefly, but there was an issue with the code that meant I had 
to revert it.

Original comment by DRPow...@gmail.com on 16 Dec 2011 at 12:38

GoogleCodeExporter commented 9 years ago
Dave, did the issue get resolved and the code redeployed?

Original comment by wjmo...@gmail.com on 22 Apr 2012 at 11:16

GoogleCodeExporter commented 9 years ago

Original comment by Matthew.Fallshaw on 3 Aug 2012 at 6:05

GoogleCodeExporter commented 9 years ago
The issue with the original code was pointed out by Dave:

https://github.com/tricycle/lesswrong/commit/b37a2157f6fe8d7bca0031696365ca536ec
e7f57

This problem needs to be addressed to finish this task.

Original comment by wjmo...@gmail.com on 15 Aug 2012 at 4:17

GoogleCodeExporter commented 9 years ago

Original comment by wjmo...@gmail.com on 15 Aug 2012 at 4:19