Multiple Shoutboxes on One Site

NodeBB-Community / nodebb-plugin-shoutbox

NodeBB Shoutbox plugin

MIT License

31 stars 34 forks source link

Multiple Shoutboxes on One Site #53

Open shaneedwards opened 10 years ago

shaneedwards commented 10 years ago

I love your plugin, and my very young community is finding it handy as well. In this case we have 4-5 distinct groups of people who are looking for this exact functionality but would like to have 4-5 shoutboxes in the header of the site with different names for each.

BreakMeta

This is a League of Legends community and they want a shoutbox for each of the major player regions around the world like North America, EU, Korea, etc.

Thanks!

Schamper commented 10 years ago

So what's the issue?

shaneedwards commented 10 years ago

Intended as more of a feature request (unless I missed the ability to do it already).

On Jul 27, 2014, at 4:35 PM, Erik Schamper notifications@github.com wrote:

So what's the issue?

— Reply to this email directly or view it on GitHub.

Schamper commented 10 years ago

Just thought about it some more (very tired, excuse me), and I now see what the issue would be. Unfortunately this would require a lot of work for a very specific use case, so I probably won't support this, not anytime soon anyway. Sorry.

Schamper commented 10 years ago

Just throwing ideas here for when I will have the time to implement this:

Admin needs to be able to somehow adjust the "Shoutbox ID" of a widget
- (Need to verify) Don't think this is possible with the current widget implementation
- Probably need to do this on the ACP page of the Shoutbox
Shouts will be stored under this selected ID e.g. shouts:id:sid
Existing shouts in the database will need to be moved.
Client side will have to be fully refactored to support this, events might be tricky.

Possible issues:

Shoutbox settings, do they apply for all the shoutbox instances or are they individual for each shoutbox instance? E.g. allow certain features in shoutbox 1 but not in shoutbox 2.
- This also applies for the muted users of a certain user, as well as banned users.

Will comment with further ideas when I think of them.

shaneedwards commented 10 years ago

Awesome! Thanks for considering it.

If it would make a difference, another way to do it would be a dropdown (or other selector) in the current shoutbox to select which box you want to view/interact with. The data storage and muting side would still have to change, but that might make the front-end implementation a bit easier to deal with.

hholst80 commented 10 years ago

Your recent fixes are not available from npm install nodebb-plugin-shoutbox? Did you forget to upload it to npm central repository? :P

Schamper commented 10 years ago

@hholst80 I didn't forget, it's just that I haven't had the time yet to properly test on 0.5.0, and I secretly still want to implement a couple more features before a new release ;)

Latest repo version should be compatible with 0.5.0 though.

faizanzahiddeloitte commented 7 years ago

Thanks for your awesome work @Schamper .... Is shoutbox working fine with Nodebb 1.6 update? Can we have support for multiple shoutboxes ,emo support and uploading pics in public chat? Please...

Schamper commented 7 years ago

Hey @faizanzahiddeloitte! You can install it on NodeBB 1.6 and report back if it works or not, and if there are any specific error messages you're seeing. As for all those features, feel free to submit a pull request and I'll gladly merge it into the main repo and publish it on npm. I currently don't have the time to work on such large features myself.