Discussion cta

# Open: Discussion CTA

**Reason:** Testing whether people would engage / enjoy in a discussion at the end of an article
**Avoid:** Taking up all the space at the bottom of an article.

With these two things in mind we have implemented (when referring to discussion bar below, we mean comment box, "discussion is closed" message or sign in bar):

* Show max 2 comment threads on default, with one discussion bar
* If there are more than 2 comments, we have a "view more" CTA
* On "viewing more" a user will be presented with maximum 8 more comment threads and another discussion bar

# Notes

* We have refactored the discussion JS module. It was becoming un-maintainable. This is a WIP.
* We are still technically still loading al the content for the first 10 comment threads. After discussions with the NextGen team, this is something that needs to be addressed, but not integral for launch.
* On this note work is being done on our API and the NextGen App to load fewer comments, and also throttle reply counts. This is almost ready.
* There is a few rendering gotchas, most of this will be dealt with once we move to our own stack that has reference to cookies from Identity

# Tech debt
* Remove old discussion modules.
* Retrospective JS unit tests (this was left out as time didn't permit)

# Future

In the future this slot is probably going to be used more for Picked Comments / Important comments or basically anything we think is important as opposed to everything. Sounds good dunit.

Let the commenting begin
![User power!](http://25.media.tumblr.com/7eedcfa0b0cc06321a840a3c5fa71b4b/tumblr_mu02erJkia1rk6p9mo1_500.gif)