-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
fix(Omnichannel): Contact Center sort by date #32566
Conversation
Looks like this PR is ready to merge! 🎉 |
🦋 Changeset detectedLatest commit: 1ebc017 The changes in this PR will be included in the next version bump. This PR includes changesets to release 32 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #32566 +/- ##
===========================================
- Coverage 56.52% 55.06% -1.47%
===========================================
Files 2484 2397 -87
Lines 54691 53207 -1484
Branches 11287 10946 -341
===========================================
- Hits 30916 29298 -1618
- Misses 21126 21319 +193
+ Partials 2649 2590 -59
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a description and a changeset to the PR?
Also, it would be awesome to have a UI test covering the solution, do you think it's possible to add it?
…ketChat/Rocket.Chat into fix/omnichannel-contact-sort-by-date
About the UI test: It's very expensive to create another test than the API integration one and there is no necessity of testing in the UI the same thing was tested from integration (if the API result is right). The fix here is not in the UI, but the API not receiving the right query string parameters. |
Co-authored-by: Marcos Spessatto Defendi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Proposed changes (including videos or screenshots)
Fix the sorting by last chat in Contact Center table.
Issue(s)
Steps to test or reproduce
Further comments
SUP-578