[FIX] Chat Transfer Failure due to Unrecognized stage name departmentId

[Shailesh351]

Proposed changes (including videos or screenshots)

This PR fixes typo in the collection name and modifies aggregator query to pass the customFilter properly

Issue(s)

https://open.rocket.chat/channel/omnichannel/thread/sxL6dbZBDoAoYJDqN

Because of extraQuery(customFilter) is being passed as a stage in aggregator here, It is giving MongoError: Unrecognized pipeline stage name: 'departmentId'

Steps to test or reproduce

Call modify.getUpdater().getLivechatUpdater().transferVisitor from RC App

Further comments

[murtaza98]

Are you sure this was required? TBH, I don't think this is required because customFilters are getting defined by this callback and as you can see here and here both these return values apply a $match filter

[Shailesh351]

When I logged what we are receiving in this function, It was showing {departmentId: 'string'}.

FYI I don't have enterprise edition, so /ee/app/livechat-enterprise/server/hooks/applySimultaneousChatsRestrictions.ts hook won't be called.

Maybe the issue is only with a non-enterprise server?

[murtaza98]

Okay got it. I found out that this was happening because livechat.applySimultaneousChatRestrictions callback was simply returning the first param passed to it incase the callback was not found, which happens in the community edition.

So I think the correct way to use this callback was to pass the params/constants in the second param and keep the first param empty/default.

PS: I took the liberty of raising a PR for the same #22839

[Shailesh351]

Thanks @murtaza98 🙃

[Shailesh351]

Closing in favor of #22839