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

[IMPROVE] Use PaginatedSelectFiltered in department edition #23054

Merged
merged 5 commits into from
Sep 3, 2021

Conversation

murtaza98
Copy link
Contributor

@murtaza98 murtaza98 commented Aug 27, 2021

Proposed changes (including videos or screenshots)

Issue(s)

When configuring the Livechat departments, not all the channels get listed in the dropdown list of channels available for offline messages to be sent.

Steps to test or reproduce

  1. Create over 10 channels (private/public) in the RC server.
  2. Navigate to Omnichannel > Departments > select any department to be edited
  3. Select the dropdown in the field "Send this department's Livechat offline messages to channel"

Expected Behavior:
After clicking on the dropdown, you should see a full list of all the channels created on the server and they're available to be selected.

Actual Behavior:
The dropdown lists just the first 10 channels on the server and searching for any channel (not on that list) returns an empty result.

Further comments

@tiagoevanp tiagoevanp changed the title [FIX] only first 10 channels getting listed in channels autocomplete [IMPROVE] Use PaginatedSelectFiltered in department edition Aug 31, 2021
@KevLehman KevLehman merged commit 995c558 into develop Sep 3, 2021
@KevLehman KevLehman deleted the fix-channel-autocomplete branch September 3, 2021 17:19
gabriellsh added a commit that referenced this pull request Sep 9, 2021
…Rocket.Chat into feat/seats-cap-request-seats

* 'feat/seats-cap-request-seats' of github.com:RocketChat/Rocket.Chat: (59 commits)
  Chore: client endpoints typings (#23152)
  Chore: Update pino and pino-pretty (#23157)
  Bump @rocket.chat/string-helpers in /ee/server/services (#23138)
  [FIX] Inaccurate use of 'Mobile notifications' instead of 'Push notifications' in i18n strings (#22978)
  Bump xml-crypto from 2.1.2 to 2.1.3 (#23141)
  [IMPROVE] Change log format to JSON (#22975)
  Regression: Auth banner for EE (#23091)
  [IMPROVE][APPS] Return task ids when using the scheduler api (#23023)
  [FIX] Update bugsnag package (#23104)
  Bump pm2 from 5.1.0 to 5.1.1 in /ee/server/services (#23128)
  Bump @types/ejson from 2.1.2 to 2.1.3 in /ee/server/services (#23126)
  Chore: Remove non-used dependencies (#23109)
  Bump @types/ws from 7.4.6 to 7.4.7 in /ee/server/services (#23095)
  Bump tar from 6.1.0 to 6.1.11 in /ee/server/services (#23068)
  Bump @types/imap from 0.8.34 to 0.8.35 (#23122)
  Bump csv-parse from 4.16.0 to 4.16.3 (#23120)
  Language update from LingoHub 🤖 (#23123)
  Bump juice from 5.2.0 to 8.0.0 (#22177)
  [FIX] Change way emails are validated on livechat registerGuest method (#23089)
  [IMPROVE] Use PaginatedSelectFiltered in department edition (#23054)
  ...
gabriellsh added a commit that referenced this pull request Sep 14, 2021
…to feat/seats-cap-info-card

* 'feat/seats-cap' of github.com:RocketChat/Rocket.Chat: (59 commits)
  Chore: client endpoints typings (#23152)
  Chore: Update pino and pino-pretty (#23157)
  Bump @rocket.chat/string-helpers in /ee/server/services (#23138)
  [FIX] Inaccurate use of 'Mobile notifications' instead of 'Push notifications' in i18n strings (#22978)
  Bump xml-crypto from 2.1.2 to 2.1.3 (#23141)
  [IMPROVE] Change log format to JSON (#22975)
  Regression: Auth banner for EE (#23091)
  [IMPROVE][APPS] Return task ids when using the scheduler api (#23023)
  [FIX] Update bugsnag package (#23104)
  Bump pm2 from 5.1.0 to 5.1.1 in /ee/server/services (#23128)
  Bump @types/ejson from 2.1.2 to 2.1.3 in /ee/server/services (#23126)
  Chore: Remove non-used dependencies (#23109)
  Bump @types/ws from 7.4.6 to 7.4.7 in /ee/server/services (#23095)
  Bump tar from 6.1.0 to 6.1.11 in /ee/server/services (#23068)
  Bump @types/imap from 0.8.34 to 0.8.35 (#23122)
  Bump csv-parse from 4.16.0 to 4.16.3 (#23120)
  Language update from LingoHub 🤖 (#23123)
  Bump juice from 5.2.0 to 8.0.0 (#22177)
  [FIX] Change way emails are validated on livechat registerGuest method (#23089)
  [IMPROVE] Use PaginatedSelectFiltered in department edition (#23054)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants