Refactor handlePriorityChange to use a local function within useMemo

Author: ggazzo

apps/meteor/client/omnichannel/hooks/useOmnichannelPrioritiesMenu.tsx

@@ -18,17 +18,17 @@ export const useOmnichannelPrioritiesMenu = (rid: IRoom['_id']) => {
 	const removeRoomPriority = useEndpoint('DELETE', '/v1/livechat/room/:rid/priority', { rid });
 	const { data: priorities } = useOmnichannelPriorities();
 
-	const handlePriorityChange = useEffectEvent((priorityId: string) => async () => {
-		try {
-			priorityId ? await updateRoomPriority({ priorityId }) : await removeRoomPriority();
-			queryClient.invalidateQueries({ queryKey: ['current-chats'] });
-			queryClient.invalidateQueries({ queryKey: roomsQueryKeys.info(rid) });
-		} catch (error) {
-			dispatchToastMessage({ type: 'error', message: error });
-		}
-	});
-
 	return useMemo(() => {
+		const handlePriorityChange = (priorityId: string) => async () => {
+			try {
+				priorityId ? await updateRoomPriority({ priorityId }) : await removeRoomPriority();
+				queryClient.invalidateQueries({ queryKey: ['current-chats'] });
+				queryClient.invalidateQueries({ queryKey: roomsQueryKeys.info(rid) });
+			} catch (error) {
+				dispatchToastMessage({ type: 'error', message: error });
+			}
+		};
+
 		const unprioritizedOption = {
 			id: 'unprioritized',
 			icon: PRIORITY_ICONS[LivechatPriorityWeight.NOT_SPECIFIED].iconName,
@@ -50,5 +50,5 @@ export const useOmnichannelPrioritiesMenu = (rid: IRoom['_id']) => {
 		});
 
 		return priorities.length ? [unprioritizedOption, ...options] : [];
-	}, [t, handlePriorityChange, priorities]);
+	}, [t, priorities, updateRoomPriority, removeRoomPriority, queryClient, rid]);
 };