[NEW] Livechat setting to customize ended conversation message #10108
Conversation
| type: 'string', | ||
| group: 'Livechat', | ||
| public: true, | ||
| section: 'Conversation Finished', |
There was a problem hiding this comment.
I don't think this setting should be on a new Section, it can be confusing (there are other settings that might fit here) and I don't see a reason for a section with only one setting. so I would rather put it bellow Show agent email agent email setting 😉
There was a problem hiding this comment.
Yes, you're totally right.
I made a mistake, to be honest I'm surprise reviewing this code.
Fixed.
| @@ -66,6 +66,13 @@ <h2>{{_ "Settings"}}</h2> | |||
| <textarea class="preview-settings rc-input__element" name="offlineSuccessMessage" id="offlineSuccessMessage">{{offlineSuccessMessage}}</textarea> | |||
| </div> | |||
| </fieldset> | |||
| <fieldset> | |||
| <legend>{{_ "Conversation_finished"}}</legend> | |||
There was a problem hiding this comment.
I guess we already have this i18n strings: https://github.com/RocketChat/Rocket.Chat/blob/develop/packages/rocketchat-livechat/.app/i18n/pt-BR.i18n.json#L11
There was a problem hiding this comment.
🤦♂️ my global search excludes i18n JSON files. sry 😬
the test instance I used also didn't have this translation 🤷♂️
| @@ -40,7 +41,7 @@ Template.message.helpers({ | |||
| case 'wm': | |||
| return t('Welcome', { user: this.u.username }); | |||
| case 'livechat-close': | |||
| return t('Conversation_finished'); | |||
| return (!_.isEmpty(Livechat.conversationFinishedMessage)) ? Livechat.conversationFinishedMessage : t('Conversation_finished'); | |||
There was a problem hiding this comment.
if possible I would try to not use underscore =)
sampaiodiego
changed the title
sampaiodiego
changed the title
sampaiodiego
deleted the
livechat-customization-of-the-conversation-finished
branch
@RocketChat/core
Closes #9873
This PR allows to customize the message that will be displayed at the end of the conversation in LiveChat widget.