Teleconference Possible Bug

[rcadena]

Having not had a multimode bbs before, nor used RG teleconference much, I might be off about how this is supposed to work.

Summary: the /U (show users in room) doesn't show anything if there is more than one user in the room.
The way I expect it to work, of course, is that any user in the room should show up in the list.

Here's a video of how it's working on two nodes, compiled on the HEAD of main.

The fix is simple: the check in ShowRoom only goes into the loop to show the users if People = 0, so just make it check for not zero.

Here's my branch with the fix.

[Renegade-Exodus]

Added this to the main branch. Thanks. We we tinkering with the teleconference a few years back to make it look more like an IRC chat room, but then gave up as no on seemed to use it. Feel free to keep sending those fixes you spot! Thanks!

…

On Tue, Sep 10, 2024 at 12:06 PM Robert Cadena ***@***.***> wrote: Having not had a multimode bbs before, nor used RG teleconference much, I might be off about how this is supposed to work. Summary: the /U (show users in room) doesn't show anything if there is more than one user in the room. The way I expect it to work, of course, is that any user in the room should show up in the list. Here's a video of how it's working on two nodes, compiled on the HEAD of main. teleconf-users.gif (view on web) <https://github.com/user-attachments/assets/19f3c55c-407c-47eb-b2c0-018600062bbd> The fix is simple <https://github.com/leewoodridge/RenegadeBBS/compare/main...rcadena:RenegadeBBS:teleconf-show-users?expand=1>: the check in ShowRoom only goes into the loop to show the users if People = 0, so just make it check for not zero. Here's my branch with the fix. teleconf-fix.gif (view on web) <https://github.com/user-attachments/assets/a4d8771d-2669-44ef-b70e-7e5bc1d78705> — Reply to this email directly, view it on GitHub <#4>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AA2LMU62NHLQ5HX3GRUGLADZV4KJZAVCNFSM6AAAAABN7DFUWSVHI2DSMVQWIX3LMV43ERDJONRXK43TNFXW4OZXGE3DKNBSGE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[rcadena]

cool. I noticed that it's also not quite working for /S - I'll poke around. probably similarly easy fix.

[rcadena]

actually, I guess it does, but it got kinda rewrite on top of the previous text. Ignore my previous comment.

I think the way stuff gets displayed might be a little funky, so I might mess around to see if it's my text resolution or just the ui is not moving stuff around correctly.