Chat 2 by Siropu [Paid] 2.5.2

[Siropu]

Siropu updated Chat 2 by Siropu with a new update entry:

Bug Fixes

Fixed an issue with "Conversations tab info" admin option where "Disabled" will generate an error.
Fixed an issue with editing conversation messages where message will appear as "undefined".
Fixed an issue with AI room message thread posts not using the Bot Name as author.
Fixed an issue with message edit function where the message actions will disappear upon editing.

Read the rest of this update entry...

 

[Alfre_DGR]

Esto parece suceder en conversaciones privadas y se ha solucionado en la próxima versión.


La barra de chat es un modo de visualización que se puede cambiar desde las opciones de administrador y desde la configuración de usuario del chat si se han configurado los permisos necesarios. Sin embargo, el icono de chat a la derecha solo debería aparecer en la versión móvil. ¿Qué versión del complemento estás usando?


Problema con la configuración del widget. Resuelto mediante ticket de soporte.


Debería aparecer cuando te desplazas hasta la parte superior o inferior dependiendo de la dirección del mensaje que estés utilizando.

I'm using version 2.5.2 and I can't find the option to disable the bottom bar in the admin settings to disable it for everyone but keep the chat active.

 

[Siropu]

I can't find the option to disable the bottom bar in the admin settings to disable it for everyone but keep the chat active.

ACP > Setup > Options > Chat > Display mode ("All pages" mode displays the bar). If the user has the user group permission to change the settings from Chat the front-end, they have to change it from there as well.

 

[Alfre_DGR]

ACP > Setup > Options > Chat > Display mode ("All pages" mode displays the bar). If the user has the user group permission to change the settings from Chat the front-end, they have to change it from there as well.

I have chat page only selected and it keeps showing everywhere

 

[Siropu]

I have chat page only selected and it keeps showing everywhere

It is because of what I'be wrote here: " If the user has the user group permission to change the settings from Chat the front-end, they have to change it from there as well."

 

[Alfre_DGR]

Esto se debe a lo que escribí aquí: "Si el usuario tiene el permiso del grupo de usuarios para cambiar la configuración del Chat en el front-end, también debe cambiarla desde allí".

 

[Siropu]

What abut your own admin permissions? If users don't have that set, then only you can see it on all pages and can change it from the font-end.

 

[Kirby]

\Siropu\Chat\Navigation\Tab::chat()

$params['counter'] = ' ' . \XF::app()->container()['chatUserCount'];

The space you are adding there causes 0 being displayed when there are no users in chat (which seems kinda pointless).

Is there a specific reason why your are adding this space and if not could that be removed?

\Siropu\Chat\Listener\appSetup()

$container['chatUserCount'] = function()
{
    $options = \XF::options();

    $activeUserCount = 0;

    if ($options->siropuChatNavUserCount)
    {
        $activeUserCount = \XF::repository('Siropu\Chat:User')->getActiveUserCount();

        if ($options->siropuChatGuestRoom)
        {
            $guestServiceManager = \XF::service('Siropu\Chat:Guest\Manager');
            $activeUserCount += $guestServiceManager->getActiveGuestCount();
        }
    }

    return $activeUserCount;
};

This code loads all active chat user entities on every page load to get the user count which seems kinda wasteful.

Could this be changed so

1. The counter value is cached and only updated for example once per minute
2. (Ideally) The code only queries the count from DB but does not load all user entities

?

 

[Siropu]

The space you are adding there causes 0 being displayed when there are no users in chat (which seems kinda pointless).

That's the reason it was added and it is not pointless. It could display 0 now but the counter updates in "real-time" when there are active users in the chat.

 

[Kirby]

Hmm, updating in realtime is fine but don't you think that showing 0 needlessly grabs attention (for the human eye) and thus seems bad for usability?

It's also inconsistent with other parts of XenForo, the alerts bell icon for example doesn't show a counter value of 0 but may update if there are alerts to show as does the direct messsage icon.

Therefore I'd really appreciate if this could be changed so 0 is not displayed as counter value.