Skip to content

[IMPROVE] Remove setting to show a livechat is waiting#13992

Merged
sampaiodiego merged 15 commits intodevelopfrom
remove-guest-pool-show-connecting-setting
Apr 17, 2019
Merged

[IMPROVE] Remove setting to show a livechat is waiting#13992
sampaiodiego merged 15 commits intodevelopfrom
remove-guest-pool-show-connecting-setting

Conversation

@renatobecker-zz
Copy link

@renatobecker-zz renatobecker-zz commented Apr 3, 2019

Related to -> RocketChat/Rocket.Chat.Livechat#210.

We decided to remove the "Livechat_open_inquiery_show_connecting" setting that was used to prevent livechat users from sending messages while they were waiting in the queue.
Now, if the routing method = 'Guest Pool', the livechat users will be able to send messages even when they are waiting for an available agent.

On the new client, an alert will be displayed, notifying the users that they are waiting in the queue.

@renatobecker-zz renatobecker-zz added this to the 1.0.0 milestone Apr 3, 2019
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 3, 2019 17:40 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 3, 2019 18:23 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 5, 2019 19:07 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 6, 2019 00:27 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 6, 2019 02:22 Inactive
@sampaiodiego
Copy link
Member

the default value is already false, I don't get why remove it..

just remembering this is added by @AlecTroemel in #4572 I wonder what was his use case for it.

@AlecTroemel
Copy link
Contributor

I believe my intent was to inform the client they were "in the pool", ie waiting to connect to an agent. Otherwise, they could assume they were connected to somebody, start sending messages and get frustrated with no response. I think the clients status in the pool should be communicated to them is some way... though I don't use the live chat feature of rocket chat anymore so it's just my 2 cents. Hope that helps!

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 11, 2019 13:39 Inactive
@renatobecker-zz
Copy link
Author

the default value is already false, I don't get why remove it..

@engelgabriel required to remove the setting since it will not be used anymore..

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 11, 2019 13:40 Inactive
@renatobecker-zz
Copy link
Author

Here's how the widget will look when waiting for an available agent:

55252716-48848e80-5232-11e9-8051-16e9682cb686

@sampaiodiego sampaiodiego temporarily deployed to rocket-chat-pr-13992 April 15, 2019 12:55 Inactive
Copy link
Member

@sampaiodiego sampaiodiego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you please write a migration to remove the setting Livechat_open_inquiery_show_connecting as well?

please remove the i18n key too.

@renatobecker how will it look on the old client? since it is still the default, I think it should have some status as you showed for the new client.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 17, 2019 14:25 Inactive
@renatobecker-zz
Copy link
Author

can you please write a migration to remove the setting Livechat_open_inquiery_show_connecting as well?

please remove the i18n key too.

@renatobecker how will it look on the old client? since it is still the default, I think it should have some status as you showed for the new client.

Okay, I'll create a migration to remove the deprecated setting, as well as removing the i18n keys..
Regarding the old client, I'll add some alert as we do on the new client.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 17, 2019 18:36 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 17, 2019 18:46 Inactive
@renatobecker-zz
Copy link
Author

@sampaiodiego,

I Just pushed the code adding an alert when the guest is waiting for an available agent, as displayed below(Old livechat version):

Screen Shot 2019-04-17 at 15 45 46

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 17, 2019 18:55 Inactive
sampaiodiego and others added 2 commits April 17, 2019 16:36
Co-Authored-By: renatobecker <renato.augusto.becker@gmail.com>
Co-Authored-By: renatobecker <renato.augusto.becker@gmail.com>
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13992 April 17, 2019 19:36 Inactive
@sampaiodiego sampaiodiego merged commit d132b91 into develop Apr 17, 2019
@sampaiodiego sampaiodiego deleted the remove-guest-pool-show-connecting-setting branch April 17, 2019 20:19
@sampaiodiego sampaiodiego changed the title [IMPROVE] Remove 'Livechat_open_inquiery_show_connecting' setting [IMPROVE] Remove setting to show a livechat is waiting Apr 17, 2019
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
@rasos
Copy link
Contributor

rasos commented Sep 14, 2019

There is a typo in the german translation, I corrected it in Lingohub but can't find that term in de.i18n.json

"Please_wait_for_the_next_available_agent": "Bitte auf die nächste freie Beratung warten",

I also tried to apply a manual translation but it seems the livechat gets his translations from somewhere else?

@sampaiodiego
Copy link
Member

thanks @rasos .. the new livechat widget has its own translation string at https://github.com/RocketChat/Rocket.Chat.Livechat

we should be creating a lingohub project for it soon.

@rasos
Copy link
Contributor

rasos commented Sep 16, 2019

Ah I see, the translations in the Livechat 2.0 project seem okay.

However we may be keeping the 1.0 widget for a while, we prefer the bar and not the icon. Any hints where to find that? github does not know e.g. Please_wait_for_the_next_available_agent in any RocketChat project and it seems not to be taken from Lingohub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

Comments