From 45417eadcef7a451a6737b64bbf365495449a37d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=BAlia=20Jaeger=20Foresti?=
 <60678893+juliajforesti@users.noreply.github.com>
Date: Fri, 20 May 2022 14:17:58 -0300
Subject: [PATCH] Chore: Convert apps/meteor/client/components/UserAutoComplete
 (#25554)

Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
---
 .../CreateDiscussion/CreateDiscussion.tsx     |  2 +-
 .../Omnichannel/modals/ForwardChatModal.tsx   |  3 +-
 .../components/RoomAutoComplete/Avatar.js     | 10 ------
 .../components/RoomAutoComplete/Avatar.tsx    | 16 ++++++++++
 ...omAutoComplete.js => RoomAutoComplete.tsx} | 31 ++++++++++++-------
 ...erAutoComplete.js => UserAutoComplete.tsx} | 30 +++++++++++++-----
 .../UsersInRole/UsersInRolePage.tsx           | 10 ++++--
 .../views/omnichannel/managers/AddManager.tsx |  9 +++++-
 packages/core-typings/src/IUser.ts            |  1 +
 packages/rest-typings/src/v1/users.ts         |  4 ++-
 10 files changed, 79 insertions(+), 37 deletions(-)
 delete mode 100644 apps/meteor/client/components/RoomAutoComplete/Avatar.js
 create mode 100644 apps/meteor/client/components/RoomAutoComplete/Avatar.tsx
 rename apps/meteor/client/components/RoomAutoComplete/{RoomAutoComplete.js => RoomAutoComplete.tsx} (56%)
 rename apps/meteor/client/components/UserAutoComplete/{UserAutoComplete.js => UserAutoComplete.tsx} (52%)

diff --git a/apps/meteor/client/components/CreateDiscussion/CreateDiscussion.tsx b/apps/meteor/client/components/CreateDiscussion/CreateDiscussion.tsx
index 0b684b4d1488..685b9f29fbf4 100644
--- a/apps/meteor/client/components/CreateDiscussion/CreateDiscussion.tsx
+++ b/apps/meteor/client/components/CreateDiscussion/CreateDiscussion.tsx
@@ -94,7 +94,7 @@ const CreateDiscussion = ({ onClose, defaultParentRoom, parentMessageId, nameSug
 									value={parentRoom}
 									onChange={handleParentRoom}
 									placeholder={t('Discussion_target_channel_description')}
-									disabled={defaultParentRoom}
+									disabled={Boolean(defaultParentRoom)}
 								/>
 							)}
 						</Field.Row>
diff --git a/apps/meteor/client/components/Omnichannel/modals/ForwardChatModal.tsx b/apps/meteor/client/components/Omnichannel/modals/ForwardChatModal.tsx
index 616895ddc771..853dd7c7d293 100644
--- a/apps/meteor/client/components/Omnichannel/modals/ForwardChatModal.tsx
+++ b/apps/meteor/client/components/Omnichannel/modals/ForwardChatModal.tsx
@@ -109,9 +109,8 @@ const ForwardChatModal = ({
 					<Field.Row>
 						<UserAutoComplete
 							conditions={conditions}
-							flexGrow={1}
 							placeholder={t('Username')}
-							onChange={(value: string): void => {
+							onChange={(value: any): void => {
 								setValue('username', value);
 							}}
 							value={getValues().username}
diff --git a/apps/meteor/client/components/RoomAutoComplete/Avatar.js b/apps/meteor/client/components/RoomAutoComplete/Avatar.js
deleted file mode 100644
index 859cafdc8bd9..000000000000
--- a/apps/meteor/client/components/RoomAutoComplete/Avatar.js
+++ /dev/null
@@ -1,10 +0,0 @@
-import { Options } from '@rocket.chat/fuselage';
-import React from 'react';
-
-import RoomAvatar from '../avatar/RoomAvatar';
-
-const Avatar = ({ value, type, avatarETag, ...props }) => (
-	<RoomAvatar size={Options.AvatarSize} room={{ type, _id: value, avatarETag }} {...props} />
-);
-
-export default Avatar;
diff --git a/apps/meteor/client/components/RoomAutoComplete/Avatar.tsx b/apps/meteor/client/components/RoomAutoComplete/Avatar.tsx
new file mode 100644
index 000000000000..d90fa71772b4
--- /dev/null
+++ b/apps/meteor/client/components/RoomAutoComplete/Avatar.tsx
@@ -0,0 +1,16 @@
+import { Options } from '@rocket.chat/fuselage';
+import React, { ReactElement } from 'react';
+
+import RoomAvatar from '../avatar/RoomAvatar';
+
+type AvatarProps = {
+	value: string;
+	type: string;
+	avatarETag?: string | undefined;
+};
+
+const Avatar = ({ value, type, avatarETag, ...props }: AvatarProps): ReactElement => (
+	<RoomAvatar size={Options.AvatarSize} room={{ t: type, type, _id: value, avatarETag }} {...props} />
+);
+
+export default Avatar;
diff --git a/apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.js b/apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.tsx
similarity index 56%
rename from apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.js
rename to apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.tsx
index 79c6ade0ff0a..762bd2cc7ed1 100644
--- a/apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.js
+++ b/apps/meteor/client/components/RoomAutoComplete/RoomAutoComplete.tsx
@@ -1,13 +1,22 @@
 import { AutoComplete, Option, Box } from '@rocket.chat/fuselage';
-import React, { memo, useMemo, useState } from 'react';
+import React, { ComponentProps, memo, ReactElement, useMemo, useState } from 'react';
 
 import { useEndpointData } from '../../hooks/useEndpointData';
 import RoomAvatar from '../avatar/RoomAvatar';
 import Avatar from './Avatar';
 
-const query = (term = '') => ({ selector: JSON.stringify({ name: term }) });
+const query = (
+	term = '',
+): {
+	selector: string;
+} => ({ selector: JSON.stringify({ name: term }) });
 
-const RoomAutoComplete = (props) => {
+type RoomAutoCompleteProps = Omit<ComponentProps<typeof AutoComplete>, 'value' | 'filter'> & {
+	value: any;
+};
+
+/* @deprecated */
+const RoomAutoComplete = (props: RoomAutoCompleteProps): ReactElement => {
 	const [filter, setFilter] = useState('');
 	const { value: data } = useEndpointData(
 		'rooms.autocomplete.channelAndPrivate',
@@ -15,21 +24,19 @@ const RoomAutoComplete = (props) => {
 	);
 	const options = useMemo(
 		() =>
-			(data &&
-				data.items.map(({ name, _id, avatarETag, t }) => ({
-					value: _id,
-					label: { name, avatarETag, type: t },
-				}))) ||
-			[],
+			data?.items.map(({ name, _id, avatarETag, t }) => ({
+				value: _id,
+				label: { name, avatarETag, type: t },
+			})) || [],
 		[data],
-	);
+	) as unknown as { value: string; label: string }[];
 
 	return (
 		<AutoComplete
 			{...props}
 			filter={filter}
 			setFilter={setFilter}
-			renderSelected={({ value, label }) => (
+			renderSelected={({ value, label }): ReactElement => (
 				<>
 					<Box margin='none' mi='x2'>
 						<RoomAvatar size='x20' room={{ type: label?.type || 'c', _id: value, ...label }} />{' '}
@@ -39,7 +46,7 @@ const RoomAutoComplete = (props) => {
 					</Box>
 				</>
 			)}
-			renderItem={({ value, label, ...props }) => (
+			renderItem={({ value, label, ...props }): ReactElement => (
 				<Option key={value} {...props} label={label.name} avatar={<Avatar value={value} {...label} />} />
 			)}
 			options={options}
diff --git a/apps/meteor/client/components/UserAutoComplete/UserAutoComplete.js b/apps/meteor/client/components/UserAutoComplete/UserAutoComplete.tsx
similarity index 52%
rename from apps/meteor/client/components/UserAutoComplete/UserAutoComplete.js
rename to apps/meteor/client/components/UserAutoComplete/UserAutoComplete.tsx
index dc134617260f..f24bbb096be3 100644
--- a/apps/meteor/client/components/UserAutoComplete/UserAutoComplete.js
+++ b/apps/meteor/client/components/UserAutoComplete/UserAutoComplete.tsx
@@ -1,13 +1,26 @@
 import { AutoComplete, Option, Box, Chip, Options } from '@rocket.chat/fuselage';
 import { useDebouncedValue } from '@rocket.chat/fuselage-hooks';
-import React, { memo, useMemo, useState } from 'react';
+import React, { ComponentProps, memo, MouseEventHandler, ReactElement, useMemo, useState } from 'react';
 
 import { useEndpointData } from '../../hooks/useEndpointData';
 import UserAvatar from '../avatar/UserAvatar';
 
-const query = (term = '', conditions = {}) => ({ selector: JSON.stringify({ term, conditions }) });
+const query = (
+	term = '',
+	conditions = {},
+): {
+	selector: string;
+} => ({ selector: JSON.stringify({ term, conditions }) });
 
-const UserAutoComplete = (props) => {
+type UserAutoCompleteProps = Omit<ComponentProps<typeof AutoComplete>, 'value' | 'filter'> &
+	Omit<ComponentProps<typeof Option>, 'value'> & {
+		conditions?: { [key: string]: unknown };
+		onChange?: MouseEventHandler<HTMLButtonElement>;
+		value: any;
+		filter?: string;
+	};
+
+const UserAutoComplete = ({ value, ...props }: UserAutoCompleteProps): ReactElement => {
 	const { conditions = {} } = props;
 	const [filter, setFilter] = useState('');
 	const debouncedFilter = useDebouncedValue(filter, 1000);
@@ -17,20 +30,21 @@ const UserAutoComplete = (props) => {
 		useMemo(() => query(debouncedFilter, conditions), [filter]),
 	);
 
-	const options = useMemo(() => (data && data.items.map((user) => ({ value: user.username, label: user.name }))) || [], [data]);
+	const options = useMemo(() => data?.items.map((user) => ({ value: user.username, label: user.name || user.username })) || [], [data]);
 
 	return (
 		<AutoComplete
 			{...props}
+			value={value}
 			filter={filter}
 			setFilter={setFilter}
-			renderSelected={({ value, label }) => {
+			renderSelected={({ value, label }): ReactElement => {
 				if (!value) {
-					return '';
+					undefined;
 				}
 
 				return (
-					<Chip height='x20' value={value} onClick={() => props.onChange()} mie='x4'>
+					<Chip height='x20' value={value} onClick={props.onChange} mie='x4'>
 						<UserAvatar size='x20' username={value} />
 						<Box verticalAlign='middle' is='span' margin='none' mi='x4'>
 							{label}
@@ -38,7 +52,7 @@ const UserAutoComplete = (props) => {
 					</Chip>
 				);
 			}}
-			renderItem={({ value, ...props }) => (
+			renderItem={({ value, ...props }): ReactElement => (
 				<Option key={value} avatar={<UserAvatar size={Options.AvatarSize} username={value} />} {...props} />
 			)}
 			options={options}
diff --git a/apps/meteor/client/views/admin/permissions/UsersInRole/UsersInRolePage.tsx b/apps/meteor/client/views/admin/permissions/UsersInRole/UsersInRolePage.tsx
index 19bea11d2547..018abbcb33c5 100644
--- a/apps/meteor/client/views/admin/permissions/UsersInRole/UsersInRolePage.tsx
+++ b/apps/meteor/client/views/admin/permissions/UsersInRole/UsersInRolePage.tsx
@@ -39,7 +39,7 @@ const UsersInRolePage = ({ role }: { role: IRole }): ReactElement => {
 			setUser(undefined);
 			reload.current?.();
 		} catch (error) {
-			dispatchToastMessage({ type: 'error', message: error });
+			dispatchToastMessage({ type: 'error', message: String(error) });
 		}
 	});
 
@@ -51,6 +51,12 @@ const UsersInRolePage = ({ role }: { role: IRole }): ReactElement => {
 		return setUser(user);
 	});
 
+	const handleChange = (value: unknown): void => {
+		if (typeof value === 'string') {
+			setRid(value);
+		}
+	};
+
 	return (
 		<Page>
 			<Page.Header title={`${t('Users_in_role')} "${description || name}"`}>
@@ -65,7 +71,7 @@ const UsersInRolePage = ({ role }: { role: IRole }): ReactElement => {
 							<Field mbe='x4'>
 								<Field.Label>{t('Choose_a_room')}</Field.Label>
 								<Field.Row>
-									<RoomAutoComplete value={rid} onChange={setRid} placeholder={t('User')} />
+									<RoomAutoComplete value={rid} onChange={handleChange} placeholder={t('User')} />
 								</Field.Row>
 							</Field>
 						)}
diff --git a/apps/meteor/client/views/omnichannel/managers/AddManager.tsx b/apps/meteor/client/views/omnichannel/managers/AddManager.tsx
index 6a4f167c96bc..58e6d956afce 100644
--- a/apps/meteor/client/views/omnichannel/managers/AddManager.tsx
+++ b/apps/meteor/client/views/omnichannel/managers/AddManager.tsx
@@ -23,12 +23,19 @@ const AddManager = ({ reload }: { reload: () => void }): ReactElement => {
 		reload();
 		setUsername('');
 	});
+
+	const handleChange = (value: unknown): void => {
+		if (typeof value === 'string') {
+			setUsername(value);
+		}
+	};
+
 	return (
 		<Box display='flex' alignItems='center' pi='x24'>
 			<Field>
 				<Field.Label>{t('Username')}</Field.Label>
 				<Field.Row>
-					<UserAutoComplete value={username} onChange={setUsername} />
+					<UserAutoComplete value={username} onChange={handleChange} />
 					<Button disabled={!username} onClick={handleSave} mis='x8' primary>
 						{t('Add')}
 					</Button>
diff --git a/packages/core-typings/src/IUser.ts b/packages/core-typings/src/IUser.ts
index c7d00ad750f6..9d0b4547d639 100644
--- a/packages/core-typings/src/IUser.ts
+++ b/packages/core-typings/src/IUser.ts
@@ -109,6 +109,7 @@ export interface IUser extends IRocketChatRecord {
 	type: string;
 	active: boolean;
 	username?: string;
+	nickname?: string;
 	name?: string;
 	services?: IUserServices;
 	emails?: IUserEmail[];
diff --git a/packages/rest-typings/src/v1/users.ts b/packages/rest-typings/src/v1/users.ts
index 1dedd4791400..0f9951f4dfec 100644
--- a/packages/rest-typings/src/v1/users.ts
+++ b/packages/rest-typings/src/v1/users.ts
@@ -10,7 +10,9 @@ export type UsersEndpoints = {
 		POST: (params: { emailOrUsername: string }) => void;
 	};
 	'users.autocomplete': {
-		GET: (params: { selector: string }) => { items: IUser[] };
+		GET: (params: { selector: string }) => {
+			items: Required<Pick<IUser, '_id' | 'name' | 'username' | 'nickname' | 'status' | 'avatarETag'>>[];
+		};
 	};
 	'users.listTeams': {
 		GET: (params: { userId: IUser['_id'] }) => { teams: Array<ITeam> };