Chore: Manager Page Rewrite (#25431)

Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
RocketChat · May 9, 2022 · 462832a · 462832a
1 parent 72100e8
commit 462832a
Show file tree

Hide file tree

Showing 6 changed files with 166 additions and 190 deletions.
diff --git a/.../views/omnichannel/managers/AddManager.js → ...views/omnichannel/managers/AddManager.tsx b/.../views/omnichannel/managers/AddManager.js → ...views/omnichannel/managers/AddManager.tsx
@@ -1,14 +1,14 @@
 import { Button, Box, Field } from '@rocket.chat/fuselage';
 import { useMutableCallback } from '@rocket.chat/fuselage-hooks';
-import React, { useState } from 'react';
+import React, { useState, ReactElement } from 'react';
 
 import UserAutoComplete from '../../../components/UserAutoComplete';
 import { useTranslation } from '../../../contexts/TranslationContext';
 import { useEndpointAction } from '../../../hooks/useEndpointAction';
 
-function AddManager({ reload, ...props }) {
+const AddManager = ({ reload }: { reload: () => void }): ReactElement => {
 	const t = useTranslation();
-	const [username, setUsername] = useState();
+	const [username, setUsername] = useState('');
 
 	const saveAction = useEndpointAction('POST', 'livechat/users/manager', { username });
 
@@ -17,14 +17,14 @@ function AddManager({ reload, ...props }) {
 			return;
 		}
 		const result = await saveAction();
-		if (!result.success) {
+		if (!result?.success) {
 			return;
 		}
 		reload();
-		setUsername();
+		setUsername('');
 	});
 	return (
-		<Box display='flex' alignItems='center' {...props}>
+		<Box display='flex' alignItems='center' pi='x24'>
 			<Field>
 				<Field.Label>{t('Username')}</Field.Label>
 				<Field.Row>
@@ -36,6 +36,6 @@ function AddManager({ reload, ...props }) {
 			</Field>
 		</Box>
 	);
-}
+};
 
 export default AddManager;
diff --git a/apps/meteor/client/views/omnichannel/managers/ManagersPage.js b/apps/meteor/client/views/omnichannel/managers/ManagersPage.js
diff --git a/apps/meteor/client/views/omnichannel/managers/ManagersRoute.js b/apps/meteor/client/views/omnichannel/managers/ManagersRoute.js
diff --git a/apps/meteor/client/views/omnichannel/managers/ManagersRoute.tsx b/apps/meteor/client/views/omnichannel/managers/ManagersRoute.tsx
@@ -0,0 +1,132 @@
+import { Box, Pagination } from '@rocket.chat/fuselage';
+import { useDebouncedValue } from '@rocket.chat/fuselage-hooks';
+import React, { ReactElement, useMemo } from 'react';
+
+import {
+	GenericTable,
+	GenericTableBody,
+	GenericTableCell,
+	GenericTableHeader,
+	GenericTableHeaderCell,
+	GenericTableLoadingTable,
+	GenericTableRow,
+} from '../../../components/GenericTable';
+import { usePagination } from '../../../components/GenericTable/hooks/usePagination';
+import { useSort } from '../../../components/GenericTable/hooks/useSort';
+import Page from '../../../components/Page';
+import UserAvatar from '../../../components/avatar/UserAvatar';
+import { usePermission } from '../../../contexts/AuthorizationContext';
+import { useTranslation } from '../../../contexts/TranslationContext';
+import { useEndpointData } from '../../../hooks/useEndpointData';
+import { AsyncStatePhase } from '../../../lib/asyncState';
+import NotAuthorizedPage from '../../notAuthorized/NotAuthorizedPage';
+import AddManager from './AddManager';
+import RemoveManagerButton from './RemoveManagerButton';
+
+const ManagersRoute = (): ReactElement => {
+	const { sortBy, sortDirection, setSort } = useSort<'name' | 'username' | 'emails.address'>('name');
+	const t = useTranslation();
+
+	const { current, itemsPerPage, setItemsPerPage: onSetItemsPerPage, setCurrent: onSetCurrent, ...paginationProps } = usePagination();
+
+	const query = useDebouncedValue(
+		useMemo(
+			() => ({
+				// text,
+				fields: JSON.stringify({ name: 1, username: 1, emails: 1, avatarETag: 1 }),
+				sort: `{ "${sortBy}": ${sortDirection === 'asc' ? 1 : -1} }`,
+				count: itemsPerPage,
+				offset: current,
+			}),
+			[itemsPerPage, current, sortBy, sortDirection],
+		),
+		500,
+	);
+
+	const { reload, ...result } = useEndpointData('livechat/users/manager', query);
+	const canViewManagers = usePermission('manage-livechat-managers');
+
+	if (!canViewManagers) {
+		return <NotAuthorizedPage />;
+	}
+
+	return (
+		<Page flexDirection='row'>
+			<Page>
+				<Page.Header title={t('Managers')} />
+				<AddManager reload={reload} />
+				<Page.Content>
+					<GenericTable>
+						<GenericTableHeader>
+							<GenericTableHeaderCell key={'name'} direction={sortDirection} active={sortBy === 'name'} onClick={setSort} sort='name'>
+								{t('Name')}
+							</GenericTableHeaderCell>
+							<GenericTableHeaderCell
+								key={'username'}
+								direction={sortDirection}
+								active={sortBy === 'username'}
+								onClick={setSort}
+								sort='username'
+							>
+								{t('Username')}
+							</GenericTableHeaderCell>
+							<GenericTableHeaderCell
+								key={'email'}
+								direction={sortDirection}
+								active={sortBy === 'emails.address'}
+								onClick={setSort}
+								sort='emails.address'
+							>
+								{t('Email')}
+							</GenericTableHeaderCell>
+							<GenericTableHeaderCell key={'remove'} w='x60'>
+								{t('Remove')}
+							</GenericTableHeaderCell>
+						</GenericTableHeader>
+						<GenericTableBody>
+							{result.phase === AsyncStatePhase.LOADING && <GenericTableLoadingTable headerCells={2} />}
+							{result.phase === AsyncStatePhase.RESOLVED &&
+								result.value.users.length > 0 &&
+								result.value.users.map((user) => (
+									<GenericTableRow key={user._id} tabIndex={0} qa-user-id={user._id}>
+										<GenericTableCell withTruncatedText>
+											<Box display='flex' alignItems='center'>
+												<UserAvatar size='x28' username={user.username || ''} etag={user.avatarETag} />
+												<Box display='flex' withTruncatedText mi='x8'>
+													<Box display='flex' flexDirection='column' alignSelf='center' withTruncatedText>
+														<Box fontScale='p2m' withTruncatedText color='default'>
+															{user.name || user.username}
+														</Box>
+													</Box>
+												</Box>
+											</Box>
+										</GenericTableCell>
+										<GenericTableCell>
+											<Box fontScale='p2m' withTruncatedText color='hint'>
+												{user.username}
+											</Box>
+											<Box mi='x4' />
+										</GenericTableCell>
+										<GenericTableCell withTruncatedText>{user.emails?.length && user.emails[0].address}</GenericTableCell>
+										<RemoveManagerButton _id={user._id} reload={reload} />
+									</GenericTableRow>
+								))}
+						</GenericTableBody>
+					</GenericTable>
+					{result.phase === AsyncStatePhase.RESOLVED && (
+						<Pagination
+							current={current}
+							itemsPerPage={itemsPerPage}
+							count={result.value.total || 0}
+							onSetItemsPerPage={onSetItemsPerPage}
+							onSetCurrent={onSetCurrent}
+							{...paginationProps}
+						/>
+					)}
+				</Page.Content>
+			</Page>
+		</Page>
+	);
+};
+
+export default ManagersRoute;