[nodebb-plugin-shoutbox] Shoutbox plugin
-
Is there anything to do apart from npm install?
I enabled the plugin, restarted NodeBB, cleared cache, but can't send messages.
Also if I start NodeBB in development mode I get a warning: warn: [socket.io] Unrecognized message: api:modules.shoutbox.get
(I'm using the edge version of NodeBB, freshly pulled.) -
Ye they changed the sockets around a bit on the latest version, will get around to making my plugin compatible asap.
-
@BarveyHirdman I updated it and pushed a new version to npm, should work fine now.
0.0.3 changelog:
- Fix shout deletion on newly created shouts
- Completely refactored client shoutbox.js, much easier for me to expand, and for other people to understand!
- Now compatible with their new socketAL implementation
-
@Mr_Waffle works like a charm
-
Thanks for posting changelogs, that helps a lot. ^_^'
-
lol wow, you know I was just asking the team earlier today on how to implement websockets in one of the plugins I was working on. Clearly you're one step ahead of me @Mr_Waffle
-
@psychobunny Yeah it took me a while to find a nice way without adding anything to the core code, but I think my method is pretty nice Would be even better if there was a
plugin
socket.io module, where all the plugins would hook into -
the way you did it is interesting. I was asking julian earlier to implement a hook in websockets so we could add new socket calls (similar to the one in webserver for routes). maybe we don't need it then... @julian thoughts?
-
I just wrote this and used the same method as @mr_waffle. This seems fine to me and it also makes me think if we could use the same system in our other plugins. For example getting the express
app
object and adding routes to it inside the plugin instead of putting a hook forcreate_routes
. -
we need to ditch the
$(document).bind('DOMNodeInserted', function(event) {
and replace that with an
ajaxify.onchange
event for 0.3.1Just a heads up, for the three people who are using that technique atm
p.s. I have no idea how badly that performs but I would imagine it sucks.
-
@psychobunny Yup, requested that in the hook request thread as well
-
@StuartH said:
Only when my wife is near
no on nodebb 2.1Only edge is supported right now. I suppose you can do `npm install [email protected] to still get basic shoutbox functionality on your version.
-
let me know what I got to do to fix that @Mr_Waffle
-
Working fine for me.