Add close Liveblog option #16

Closed
nmec opened this Issue Sep 5, 2012 · 10 comments

Comments

Projects
None yet
4 participants

nmec commented Sep 5, 2012

It would be good to be able to close liveblogs once the event they are covering has finished.

This would stop the polling for updates and display the liveblog entries in chronological order.

And dump the generated entries as HTML so that the content is preserved, even upon plugin deletion. Please? :)

norcross commented Sep 6, 2012

hmm...I see that each entry is stored in the comments table. both the comment_approved and comment_type are 'liveblog'. So I'd imagine that a function (separate shortcode?) could handle the output. Lemme whip something up.

norcross commented Sep 6, 2012

Here's a rough proof-of-concept that would work. https://gist.github.com/3659837

while I don't like the idea of using an actual SQL query to handle it, running the get_comments won't work using the liveblog entries. Also, I see the main class uses a SQL as well, so I figured it would be OK.

Owner

nb commented Sep 18, 2012

@norcross, @kovshenin I think it would be enough to stop polling on the client-side. No need for separate recap functionality.

Apart from that, the blog authors can change the post content to communicate that the liveblog is over.

@nb the issue is that once it's done polling, the content that came from the liveblog is no longer visible. Many times sites want to show that. It's certainly something that can be made as an add-on and not included in the plugin itself.

Owner

nb commented Sep 18, 2012

@norcross, if you don't remove the "This is a liveblog" checkbox, it will be visible. On page load we serve all current entries. If we don't poll for more they just won't change.

@nb ahh, I misunderstood. That would work perfectly.

@nb nb referenced this issue Oct 11, 2012

Closed

Archiving liveblogs #43

@ghost ghost assigned nb Dec 20, 2012

@nb nb referenced this issue Dec 20, 2012

Merged

Archiving a Liveblog #60

Owner

nb commented Dec 20, 2012

In pull request #60 there's an implementation, feedback is welcome.

Owner

nb commented Dec 26, 2012

Fixed in #60.

@nb nb closed this Dec 26, 2012

Owner

nb commented Dec 26, 2012

Actually fixed in 4089398.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment