fix: review

RocketChat · Jan 2, 2024 · d8288b8 · d8288b8
1 parent 5db7320
commit d8288b8
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/apps/meteor/client/startup/actionButtons/pinMessage.ts b/apps/meteor/client/startup/actionButtons/pinMessage.ts
@@ -20,29 +20,28 @@ Meteor.startup(() => {
 		context: ['pinned', 'message', 'message-mobile', 'threads', 'direct', 'videoconf', 'videoconf-threads'],
 		async action(_, props) {
 			const { message = messageArgs(this).msg } = props;
-			const onCloseModal = () => {
-				imperativeModal.close();
-			};
-			const onConfirm = async (): Promise<void> => {
+
+			const handleConfirm = async () => {
 				message.pinned = true;
 				try {
 					await sdk.call('pinMessage', message);
 					queryClient.invalidateQueries(['rooms', message.rid, 'pinned-messages']);
 				} catch (error) {
 					dispatchToastMessage({ type: 'error', message: error });
 				}
-				onCloseModal();
+				imperativeModal.close();
 			};
 
 			imperativeModal.open({
 				component: GenericModal,
 				props: {
 					title: 'Are you sure you want to pin this message?',
-					variant: 'info',
+					variant: 'warning',
+					icon: null,
 					confirmText: 'Yes pin it',
-					onConfirm,
-					onClose: onCloseModal,
-					onCancel: onCloseModal,
+					onConfirm: handleConfirm,
+					onClose: () => imperativeModal.close(),
+					onCancel: () => imperativeModal.close(),
 				},
 			});
 		},