Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEW] Omnichannel Contact Center (Directory) #19931

Merged
merged 9 commits into from
Dec 22, 2020

Conversation

rafaelblink
Copy link
Contributor

@rafaelblink rafaelblink commented Dec 22, 2020

Proposed changes

Omnichannel Contact Center (or Directory) is a way that the agents for example can see more details about chat conversations, they can create new contacts.

As an agent, for example, you can see chat duration of your conversations, you can assign e-mail address and phone to your contacts, if you are an enterprise, you can assign Rocket.Chat users as a contact manager, and so on.

Issue(s)

How to test or reproduce

Screenshots

image

image

image

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Hotfix (a major bugfix that has to be merged asap)
  • Documentation Update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Changelog

Further comments

rafaelblink and others added 5 commits November 27, 2020 15:07
* [FEATURE] Directory page starting contacts/visitors tab.

* [IMPROVE] Removing unused variable.

* [IMPROVE] Directory link tooltip to just "Directory" instead "Omnichannel Directory"

* Implementing improves in contact tab, routes, last chat column mock and new button aside filter

* Contextual bar and routes for info and new contacts

* Contact endpoints for omnichannel directory forms.

* "Last Chat" missing translation

* Omnichannel Directory - Create new contact form.

* view a specific contact and improves at contact form.

* "edit" contact form with same "create" component

* Show custom fields in Contact Info page.

* handling custom fields on edit/create contacts.

* Showing only "custom fields" that are visible and with "visitor" scope.

* CreatedAt data in Contact/Visitor and ContactInfo.

* Add LastChat data in Contact/Visitor.

* Show LastChat in Contact Info page and Grid

* Email validation improvement and asterisk on required field

* Starting "contact manager" on edit/create page.

* show contact manager user info

* closing contextbar when you click at cancel button, removing chat history button and fixing contact manager field.

* Change useEndpointDataExperimental to useEndpointData.

* Custom fields required with asterisk.

* just using the LivechatVisitors.findOneById method

* changing POST method to use existing method to save visitors/contacts, renaming contactManager data.

* PUT not accepted so we'll use POST to save and update contacts.

* Set last chat when create a new room.

* Changing place of updateLastChat method.

* Using correct translation aliases.

* Contact Manager Info to EE folder and renaming ContactForm.

* Moving EE contact manager input to proper path.

* verify if user has authorization to see contacts page.

* Fixing blank return useless.

* using callback to lastChat

* Add small improvements to the code.

Co-authored-by: Renato Becker <renato.augusto.becker@gmail.com>
@renatobecker renatobecker added this to the 3.10.0 milestone Dec 22, 2020
* [FEATURE] Directory page starting contacts/visitors tab.

* [IMPROVE] Removing unused variable.

* [IMPROVE] Directory link tooltip to just "Directory" instead "Omnichannel Directory"

* Creating omnichannel directory - chat tab page.

* Get logged user and passing to query.

* Navigate to conversation page on click at row.

* Missing translation of "Chat duration"

* Contact name column translation

* Sorting by "Closed At" column and adding tags below "Contact Name"

* Responsive tags in Chat tab

* Add permission to see chats tab and removing useless comment.

* changes after merge with develop

* after merge

* Revert "after merge"

This reverts commit 8e1860f.

* Fix blank line missing.

* showing chat tab.

Co-authored-by: Renato Becker <renato.augusto.becker@gmail.com>
@rafaelblink rafaelblink marked this pull request as ready for review December 22, 2020 04:16
@renatobecker renatobecker changed the title [FEATURE] Omnichannel Contact Center (Directory) [NEW] Omnichannel Contact Center (Directory) Dec 22, 2020
@renatobecker
Copy link
Contributor

renatobecker commented Dec 22, 2020

@rafaelblink please, improve the PR description, we need to share more details on this important feature.
Feel free to add more screenshots as well, they are very important for other community members to understand what's going to be delivered and how to use it.
Also, the documentation related to the new REST API endpoint is missing.

Thanks.

@renatobecker renatobecker added the documentation: needed Needs addition/changes to documentation label Dec 22, 2020
renatobecker
renatobecker previously approved these changes Dec 22, 2020
@RocketChat RocketChat deleted a comment from lgtm-com bot Dec 22, 2020
@RocketChat RocketChat deleted a comment from lgtm-com bot Dec 22, 2020
@lgtm-com
Copy link

lgtm-com bot commented Dec 22, 2020

This pull request introduces 2 alerts when merging e6412fd into cd092f4 - view on LGTM.com

new alerts:

  • 2 for Useless conditional

@sampaiodiego sampaiodiego merged commit e33ba54 into develop Dec 22, 2020
@sampaiodiego sampaiodiego deleted the omnichannel/directory branch December 22, 2020 05:57
@sampaiodiego
Copy link
Member

Please fix #19931 (comment) LGTM bot comments

@sampaiodiego sampaiodiego mentioned this pull request Dec 29, 2020
@yash-rajpal
Copy link
Member

@renatobecker If documentation is needed, I can try and understand the endpoint and changes and try the docs part, can you please guide me through it?
Thanks.

@renatobecker
Copy link
Contributor

@renatobecker If documentation is needed, I can try and understand the endpoint and changes and try the docs part, can you please guide me through it?
Thanks.

Hi @yash-rajpal, the team has already started working on it, but thanks for being willing to help us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation: needed Needs addition/changes to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants