Add back real-time list of people viewing thread?

<baris>

The way we did that was not scalable at all and had many issues when there were hundreds of people viewing a single thread.

I started a plugin for it https://github.com/barisusakli/nodebb-plugin-browsing-users. Waiting on a PR to be merged in to socket.io-redis so I can complete it.

Refactor requests between nodes and add `clientRooms` method by darrachequesne · Pull Request #146 · socketio/socket.io-redis-adapter

This PR refactors the way requests are sent between nodes. The subJson client has been removed, the subClient now listens to three channels by default: the usual channel, where the message are bro...

GitHub (github.com)

po

Wow, you guys are so fast at replying. Thank you for letting me know where the status is. Do you know how long it typically takes for a PR like that to be merged into socket.io-redis? I see that there are 11 open PR's right now, some of which are quite old. Is there anything I can do to help accelerate the process, e.g., by commenting to show that more people find this helpful?

After the PR merges in, when might you estimate to finish the plugin? (Also, would I need to upgrade NodeBB to a development or testing branch, or would it work with a stable release like v1.3.0?) I'm asking because I'm trying to gauge whether to inform this potential community of enthusiast educators that this will likely be released soon. They're trying to decide on which platform to use for their collaboration to organize educational material around a new set of learning standards, and I would like to propose for them to try using NodeBB + Expii instead of their existing combination of Twitter + Discussion Groups + Shared Documents.

I think that the real-time features would push this over the tipping point, as there are very few other systems with the same real sense of community. I remember how wowed I was when I first saw it myself, and I would be thrilled to introduce more and more groups to your platform technology.

julian

Definitely. It was one of my favourite features as well, and if we had it, we would definitely bundle it with every install again (provided it doesn't raise server-side resource usage too much!)

po

Thank you for your quick reply, and for sharing your opinion on the feature. It really helps my confidence to know that you value it highly. I think that it can fundamentally transform the landscape of several communities with which I am now working. I will tell them that it's coming back soon, and even speak of it as a feature to count on.

Speaking of real-time, your system certainly works well in notifying me that there was a response to this thread. It is so neat to sense that a community is alive.

julian

@po

The PR got merged! https://github.com/socketio/socket.io-redis/pull/146

po

Hooray!

Thanks for letting me know! Over the weekend, we have attracted a number of groups who are looking at our NodeBB-powered forum with significant interest. The feedback has been uniformly enthusiastic, and people have not seen anything like this before!

Will this plugin be compatible with NodeBB v1.3.0, or will we need to wait for the next release? I will install it as soon as it becomes available and stable, as I think it is immensely impactful.

Thank you for working on this!

julian

@po The pull request was for a dependency that NodeBB relies on, so it will need to be updated on our end, and would be released in the next version of NodeBB (or the one after, if we release a version today)

po

Thanks for your quick reply! I had not been following your release schedule closely. When might you anticipate that release happening? I don't mean to rush you, and will use that information to decide when to inform our groups.

julian

How soon this can be done really depends on the champion for the feature, @baris

<baris>

This is blocked until I can figure out the issues with the socket.io dependencies. https://github.com/socketio/socket.io-redis/pull/155

po

Thanks @baris for working on this!

po

Hi guys! I have been out working on other aspects of Expii for several months, and am coming back to our NodeBB installation. Time to upgrade to your new versions.

I am curious: how far along is the support for real time display of users viewing the same thread as you?

julian

Hey @baris, that upstream PR was merged last month

<baris>

Ok published 1.0.1 of the plugin, you need to pair it up with the develop branch or wait for 1.5.0. There will be a browsingUsers array in the topic route so you can modify your template and add the list of users with the below template.

<!-- BEGIN browsingUsers -->
<li data-uid="{browsingUsers.uid}">
	<a href="{config.relative_path}/user/{browsingUsers.userslug}">
		<!-- IF browsingUsers.picture -->
		<img src="{browsingUsers.picture}" />
		<!-- ELSE -->
		<div class="user-icon" style="background-color: {browsingUsers.icon:bgColor};">{browsingUsers.icon:text}</div>
		<!-- ENDIF browsingUsers.picture -->
	</a>
</li>
<!-- END browsingUsers -->

po

Great! I'll wait until 1.5.0.

Also, as I was watching this thread with my browser open, I noticed that it didn't auto-refresh when there was a new reply. I remember the old version used to do that in real time whenever someone replied and your browser window was open. Will that feature come back too?

julian

It may be that your session disconnected from real-time after being idle (another resource-saving measure)

po

Oh, I see! And I just saw your reply appear.