Topic Events plugin

a_5mith

@trevor Brings us back round to the debate on what happens if OP is deleted I guess.

Still think if there's no replies to that topic, delete the actual topic as well, not just first post.

psychobunny

13 years ago huh, not bad. I'll look into it on the plugin's end, but maybe @a_5mith has a point re: main post deletion.

trevor

@a_5mith said:

@trevor Brings us back round to the debate on what happens if OP is deleted I guess.

Still think if there's no replies to that topic, delete the actual topic as well, not just first post.

Makes sense.

Tanner

Nice, I'm really liking the timeline format of this.

unixfox

Nice :o, it looks like Github ^^.

a_5mith

user referenced {topic} x minutes ago.

psychobunny

I didn't think about that. Cool idea. Wonder if I should make it part of this or part of a separate plugin (and then use that as a way to flesh out the API)... hmmmm

Netto Hikari

julian

Can you show @psychobunny the expanded HTML in one of those blocks?

psychobunny

I think I know what's wrong. This needs to be translated: https://github.com/psychobunny/nodebb-plugin-events/tree/master/languages

If you don't mind, if you do make the translation please send a PR

I'm not sure if I should make an individual transifex project for this, maybe I'll just accept PR's for now

Xiph

Tried installing this on 0.5.1, doesn't look like the events are showing up Nothing to see in the console, sadly.

Xiph

This'd still be really nice (as it removes the need for mods to clutter up the thread with "Moved to [x]" posts), but it seems completely abandoned any chance it'll be updated for 0.6.x?

rbeer

@Xiph
I just love to revive the dead: https://github.com/Linux-statt-Windows/nodebb-plugin-topic-events

I've changed the storing logics a bit, so every un/lock, un/pin, delete/restore and move event gets recorded; not only a single instance of each which gets overwritten by every new of the same type. action:topic.purge finally deletes all events.

Please always refer to the forked repo above, not my original one, since I'm still a git-noob and bound to mess up my own repo there, sooner or later.

This will break in 0.7.0 again, but I've already made out the differences. It'll be updated as soon as I got my head around require.js.

For now I'm working on an entry for the ThreadTools, so admins/moderators can hide the events (they still get recorded, just not shown). This will come in very handy for e.g. moderated FAQ topics, which are re-opened - updated - closed several times.

@psychobunny
Please have a closer look at https://github.com/Linux-statt-Windows/nodebb-plugin-events-topic/blob/master/threadTools.patch !
(I guess) I had to introduce new hooks in order to not break other plugins which expect the topicID on the callback, only. I'd love to see that merged into 0.7.0; would also be just consistent with the other action:topic-hooks, which already receive a data-object instead of only the tid.

psychobunny

Cool I'm more than happy to transfer the repo over to you if you wish to maintain it. As far as the NodeBB core is concerned, just submit a PR and we can discuss it there