feat: Return departmentsAllowedToForward property in livechat/config endpoint

.changeset/beige-kiwis-count.md

@@ -0,0 +1,7 @@
+---
+"@rocket.chat/meteor": minor
+"@rocket.chat/livechat": minor
+"@rocket.chat/model-typings": minor
+---
+
+Added `departmentsAllowedToForward` property to departments returned in the `livechat/config` endpoint

apps/meteor/app/livechat/server/api/lib/livechat.ts

@@ -29,23 +29,19 @@ async function findTriggers(): Promise<Pick<ILivechatTrigger, '_id' | 'actions'
 
 async function findDepartments(
 	businessUnit?: string,
-): Promise<Pick<ILivechatDepartment, '_id' | 'name' | 'showOnRegistration' | 'showOnOfflineForm'>[]> {
+): Promise<Pick<ILivechatDepartment, '_id' | 'name' | 'showOnRegistration' | 'showOnOfflineForm' | 'departmentsAllowedToForward'>[]> {
 	// TODO: check this function usage
 	return (
-		await (
-			await LivechatDepartment.findEnabledWithAgentsAndBusinessUnit(businessUnit, {
-				_id: 1,
-				name: 1,
-				showOnRegistration: 1,
-				showOnOfflineForm: 1,
-			})
-		).toArray()
-	).map(({ _id, name, showOnRegistration, showOnOfflineForm }) => ({
-		_id,
-		name,
-		showOnRegistration,
-		showOnOfflineForm,
-	}));
+		await LivechatDepartment.findEnabledWithAgentsAndBusinessUnit<
+			Pick<ILivechatDepartment, '_id' | 'name' | 'showOnRegistration' | 'showOnOfflineForm' | 'departmentsAllowedToForward'>
+		>(businessUnit, {
+			_id: 1,
+			name: 1,
+			showOnRegistration: 1,
+			showOnOfflineForm: 1,
+			departmentsAllowedToForward: 1,
+		})
+	).toArray();
 }
 
 export function findGuest(token: string): Promise<ILivechatVisitor | null> {

apps/meteor/ee/server/models/raw/LivechatDepartment.ts

@@ -28,10 +28,10 @@ declare module '@rocket.chat/model-typings' {
 		): Promise<UpdateResult>;
 		unfilteredRemove(query: Filter<ILivechatDepartment>): Promise<DeleteResult>;
 		removeParentAndAncestorById(id: string): Promise<UpdateResult | Document>;
-		findEnabledWithAgentsAndBusinessUnit(
+		findEnabledWithAgentsAndBusinessUnit<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
 			businessUnit: string,
-			projection: FindOptions<ILivechatDepartment>['projection'],
-		): Promise<FindCursor<ILivechatDepartment>>;
+			projection: FindOptions<T>['projection'],
+		): Promise<FindCursor<T>>;
 		findByParentId(parentId: string, options?: FindOptions<ILivechatDepartment>): FindCursor<ILivechatDepartment>;
 		findAgentsByBusinessHourId(businessHourId: string): AggregationCursor<{ agentIds: string[] }>;
 	}
@@ -74,19 +74,19 @@ export class LivechatDepartmentEE extends LivechatDepartmentRaw implements ILive
 		return this.updateMany({ parentId: id }, { $unset: { parentId: 1 }, $pull: { ancestors: id } });
 	}
 
-	async findEnabledWithAgentsAndBusinessUnit(
+	async findEnabledWithAgentsAndBusinessUnit<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
 		businessUnit: string,
-		projection: FindOptions<ILivechatDepartment>['projection'],
-	): Promise<FindCursor<ILivechatDepartment>> {
+		projection: FindOptions<T>['projection'],
+	): Promise<FindCursor<T>> {
 		if (!businessUnit) {
-			return super.findEnabledWithAgents(projection);
+			return super.findEnabledWithAgents<T>(projection);
 		}
 		const unit = await LivechatUnit.findOneById(businessUnit, { projection: { _id: 1 } });
 		if (!unit) {
 			throw new Meteor.Error('error-unit-not-found', `Error! No Active Business Unit found with id: ${businessUnit}`);
 		}
 
-		return super.findActiveByUnitIds([businessUnit], { projection });
+		return super.findActiveByUnitIds<T>([businessUnit], { projection });
 	}
 
 	findByParentId(parentId: string, options?: FindOptions<ILivechatDepartment>): FindCursor<ILivechatDepartment> {

apps/meteor/server/models/raw/LivechatDepartment.ts

@@ -283,23 +283,25 @@ export class LivechatDepartmentRaw extends BaseRaw<ILivechatDepartment> implemen
 		return this.updateMany({ _id: { $in: _ids } }, { $inc: { numAgents: -1 } });
 	}
 
-	findEnabledWithAgents(projection: FindOptions<ILivechatDepartment>['projection'] = {}): FindCursor<ILivechatDepartment> {
+	findEnabledWithAgents<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
+		projection: FindOptions<T>['projection'] = {},
+	): FindCursor<T> {
 		const query = {
 			numAgents: { $gt: 0 },
 			enabled: true,
 		};
-		return this.find(query, projection && { projection });
+		return this.find<T>(query, projection && { projection });
 	}
 
-	async findEnabledWithAgentsAndBusinessUnit(
+	async findEnabledWithAgentsAndBusinessUnit<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
 		_: any,
-		projection: FindOptions<ILivechatDepartment>['projection'] = {},
-	): Promise<FindCursor<ILivechatDepartment>> {
+		projection: FindOptions<T>['projection'] = {},
+	): Promise<FindCursor<T>> {
 		const query = {
 			numAgents: { $gt: 0 },
 			enabled: true,
 		};
-		return this.find(query, projection && { projection });
+		return this.find<T>(query, projection && { projection });
 	}
 
 	findOneByIdOrName(_idOrName: string, options: FindOptions<ILivechatDepartment> = {}): Promise<ILivechatDepartment | null> {
@@ -328,7 +330,10 @@ export class LivechatDepartmentRaw extends BaseRaw<ILivechatDepartment> implemen
 		return this.find(query, options);
 	}
 
-	findActiveByUnitIds(unitIds: string[], options: FindOptions<ILivechatDepartment> = {}): FindCursor<ILivechatDepartment> {
+	findActiveByUnitIds<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
+		unitIds: string[],
+		options: FindOptions<T> = {},
+	): FindCursor<T> {
 		const query = {
 			enabled: true,
 			numAgents: { $gt: 0 },
@@ -338,7 +343,7 @@ export class LivechatDepartmentRaw extends BaseRaw<ILivechatDepartment> implemen
 			},
 		};
 
-		return this.find(query, options);
+		return this.find<T>(query, options);
 	}
 
 	findNotArchived(options: FindOptions<ILivechatDepartment> = {}): FindCursor<ILivechatDepartment> {

apps/meteor/tests/data/livechat/department.ts

@@ -47,9 +47,11 @@ const createDepartmentWithMethod = (
 	{
 		allowReceiveForwardOffline = false,
 		fallbackForwardDepartment,
+		departmentsAllowedToForward,
 	}: {
 		allowReceiveForwardOffline?: boolean;
 		fallbackForwardDepartment?: string;
+		departmentsAllowedToForward?: string[];
 	} = {},
 ) =>
 	new Promise((resolve, reject) => {
@@ -70,6 +72,7 @@ const createDepartmentWithMethod = (
 							description: 'created from api',
 							allowReceiveForwardOffline,
 							fallbackForwardDepartment,
+							departmentsAllowedToForward,
 						},
 						initialAgents,
 					],
@@ -141,9 +144,11 @@ export const addOrRemoveAgentFromDepartment = async (
 export const createDepartmentWithAnOfflineAgent = async ({
 	allowReceiveForwardOffline = false,
 	fallbackForwardDepartment,
+	departmentsAllowedToForward,
 }: {
 	allowReceiveForwardOffline?: boolean;
 	fallbackForwardDepartment?: string;
+	departmentsAllowedToForward?: string[];
 }): Promise<{
 	department: ILivechatDepartment;
 	agent: {
@@ -156,6 +161,7 @@ export const createDepartmentWithAnOfflineAgent = async ({
 	const department = (await createDepartmentWithMethod(undefined, {
 		allowReceiveForwardOffline,
 		fallbackForwardDepartment,
+		departmentsAllowedToForward,
 	})) as ILivechatDepartment;
 
 	await addOrRemoveAgentFromDepartment(department._id, { agentId: user._id, username: user.username }, true);

apps/meteor/tests/end-to-end/api/livechat/11-livechat.ts

@@ -4,17 +4,24 @@ import { after, before, describe, it } from 'mocha';
 import { sleep } from '../../../../lib/utils/sleep';
 import { getCredentials, api, request, credentials } from '../../../data/api-data';
 import { createCustomField, deleteCustomField } from '../../../data/livechat/custom-fields';
-import { addOrRemoveAgentFromDepartment, createDepartmentWithAnOnlineAgent } from '../../../data/livechat/department';
+import {
+	addOrRemoveAgentFromDepartment,
+	createDepartmentWithAnOfflineAgent,
+	createDepartmentWithAnOnlineAgent,
+	deleteDepartment,
+} from '../../../data/livechat/department';
 import {
 	createVisitor,
 	createLivechatRoom,
 	makeAgentUnavailable,
 	closeOmnichannelRoom,
 	sendMessage,
 	deleteVisitor,
+	createDepartment,
 } from '../../../data/livechat/rooms';
 import { createBotAgent, getRandomVisitorToken } from '../../../data/livechat/users';
 import { removePermissionFromAllRoles, restorePermissionToRoles, updatePermission, updateSetting } from '../../../data/permissions.helper';
+import { deleteUser } from '../../../data/users.helper';
 import { IS_EE } from '../../../e2e/config/constants';
 
 describe('LIVECHAT - Utils', () => {
@@ -171,6 +178,48 @@ describe('LIVECHAT - Utils', () => {
 			expect(body.config).to.have.property('room');
 			expect(body.config.room).to.have.property('_id', newRoom._id);
 		});
+		(IS_EE ? it : it.skip)('should return list of departments with at least one agent', async () => {
+			const emptyDepartment = await createDepartment();
+			const { department: forwardDepartment, agent } = await createDepartmentWithAnOnlineAgent();
+			const { department: testDepartment, agent: agent2 } = await createDepartmentWithAnOfflineAgent({
+				departmentsAllowedToForward: [forwardDepartment._id],
+			});
+
+			const { body } = await request.get(api('livechat/config')).set(credentials);
+
+			expect(body).to.have.property('success', true);
+			expect(body).to.have.property('config');
+			expect(body.config).to.have.property('departments');
+			expect(body.config.departments).to.be.an('array').with.lengthOf.at.least(2);
+
+			expect(body.config.departments).to.not.deep.include({
+				_id: emptyDepartment._id,
+				name: emptyDepartment.name,
+				showOnRegistration: emptyDepartment.showOnRegistration,
+				showOnOfflineForm: emptyDepartment.showOnOfflineForm,
+			});
+			expect(body.config.departments).to.deep.include({
+				_id: forwardDepartment._id,
+				name: forwardDepartment.name,
+				showOnRegistration: forwardDepartment.showOnRegistration,
+				showOnOfflineForm: forwardDepartment.showOnOfflineForm,
+			});
+			expect(body.config.departments).to.deep.include({
+				_id: testDepartment._id,
+				name: testDepartment.name,
+				showOnRegistration: testDepartment.showOnRegistration,
+				showOnOfflineForm: testDepartment.showOnOfflineForm,
+				departmentsAllowedToForward: [forwardDepartment._id],
+			});
+
+			await Promise.all([
+				deleteDepartment(emptyDepartment._id),
+				deleteDepartment(forwardDepartment._id),
+				deleteDepartment(testDepartment._id),
+				deleteUser(agent.user),
+				deleteUser(agent2.user),
+			]);
+		});
 	});
 
 	describe('livechat/page.visited', () => {

packages/livechat/src/definitions/departments.d.ts

@@ -3,5 +3,7 @@ export type Department = {
 	_id: string;
 	name: string;
 	showOnRegistration?: boolean;
+	showOnOfflineForm?: boolean;
+	departmentsAllowedToForward?: string[];
 	[key: string]: unknown;
 };

packages/model-typings/src/models/ILivechatDepartmentModel.ts

@@ -52,11 +52,13 @@ export interface ILivechatDepartmentModel extends IBaseModel<ILivechatDepartment
 	updateById(_id: string, update: Partial<ILivechatDepartment>): Promise<Document | UpdateResult>;
 	updateNumAgentsById(_id: string, numAgents: number): Promise<Document | UpdateResult>;
 	decreaseNumberOfAgentsByIds(_ids: string[]): Promise<Document | UpdateResult>;
-	findEnabledWithAgents(projection?: FindOptions<ILivechatDepartment>['projection']): FindCursor<ILivechatDepartment>;
-	findEnabledWithAgentsAndBusinessUnit(
+	findEnabledWithAgents<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
+		projection?: FindOptions<ILivechatDepartment>['projection'],
+	): FindCursor<T>;
+	findEnabledWithAgentsAndBusinessUnit<T extends Partial<ILivechatDepartment> = ILivechatDepartment>(
 		_: any,
-		projection: FindOptions<ILivechatDepartment>['projection'],
-	): Promise<FindCursor<ILivechatDepartment>>;
+		projection: FindOptions<T>['projection'],
+	): Promise<FindCursor<T>>;
 	findOneByIdOrName(_idOrName: string, options?: FindOptions<ILivechatDepartment>): Promise<ILivechatDepartment | null>;
 	findByUnitIds(unitIds: string[], options?: FindOptions<ILivechatDepartment>): FindCursor<ILivechatDepartment>;
 	findActiveByUnitIds(unitIds: string[], options?: FindOptions<ILivechatDepartment>): FindCursor<ILivechatDepartment>;