fix: new responsible validation (#1001)

xgeekshq · Feb 2, 2023 · 609ec51 · 609ec51
1 parent aa63ded
commit 609ec51
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/backend/src/modules/boards/services/update.board.service.ts b/backend/src/modules/boards/services/update.board.service.ts
@@ -98,11 +98,11 @@ export default class UpdateBoardServiceImpl implements UpdateBoardServiceInterfa
 		 * - is a sub-board
 		 * - and the logged user isn't the current responsible
 		 */
-		if (boardData.users && currentResponsible.id !== newResponsible.id) {
+		if (boardData.users && String(currentResponsible.id) !== String(newResponsible.id)) {
 			if (isSubBoard) {
 				const promises = boardData.users
 					.filter((boardUser) =>
-						[getIdFromObjectId(String(currentResponsible?.id)), newResponsible.id].includes(
+						[getIdFromObjectId(String(currentResponsible?.id)), String(newResponsible.id)].includes(
 							(boardUser.user as unknown as User)._id
 						)
 					)

diff --git a/backend/src/modules/communication/applications/slack-communication.application.ts b/backend/src/modules/communication/applications/slack-communication.application.ts
@@ -71,12 +71,12 @@ export class SlackCommunicationApplication implements CommunicationApplicationIn
 		const generalText = {
 			member: (
 				boardId: string
-			) => `<!channel> In order to proceed with the retro of this month, here is the board link: \n\n
+			) => `<!here> In order to proceed with the retro of this month, here is the board link: \n\n
       ${this.config.frontendUrl}/boards/${boardId}
       `,
 			responsible: (
 				boardId: string
-			) => `<!channel> In order to proceed with the retro of this month, here is the main board link: \n\n
+			) => `<!here> In order to proceed with the retro of this month, here is the main board link: \n\n
       ${this.config.frontendUrl}/boards/${boardId}
       `
 		};

diff --git a/backend/src/modules/communication/applications/slack-merge-board.application.ts b/backend/src/modules/communication/applications/slack-merge-board.application.ts
@@ -10,7 +10,7 @@ export class SlackMergeBoardApplication implements MergeBoardApplicationInterfac
 
 	async execute(data: MergeBoardType): Promise<MergeBoardType | null> {
 		const { responsiblesChannelId, teamNumber, isLastSubBoard } = data;
-		const message = `<!channel>, The board of team ${teamNumber} is ready`;
+		const message = `<!here>, The board of team ${teamNumber} is ready`;
 		this.chatHandler.postMessage(responsiblesChannelId, message);
 
 		if (isLastSubBoard) {

diff --git a/backend/src/modules/communication/applications/slack-responsible.application.ts b/backend/src/modules/communication/applications/slack-responsible.application.ts
@@ -22,7 +22,7 @@ export class SlackResponsibleApplication implements ResponsibleApplicationInterf
 
 		const newResponsibleId = await this.usersHandler.getSlackUserIdByEmail(newResponsibleEmail);
 
-		const message = `<!channel>, <@${newResponsibleId}> is the new responsible for the team ${teamNumber}`;
+		const message = `<!here>, <@${newResponsibleId}> is the new responsible for the team ${teamNumber}`;
 
 		if (mainChannelId) {
 			await this.chatHandler.postMessage(mainChannelId, message);