Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE]: add get team endpoint #592

Merged
merged 6 commits into from
Nov 18, 2022
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion backend/src/modules/teams/schemas/team.user.schema.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export default class TeamUser {
type: Boolean,
nullable: false
})
isNewbie!: boolean;
isNewJoiner!: boolean;

@Prop({ type: SchemaTypes.ObjectId, ref: 'User', nullable: false })
user!: User | ObjectId;
Expand Down
15 changes: 13 additions & 2 deletions backend/src/modules/teams/services/get.team.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export default class GetTeamService implements GetTeamServiceInterface {
teamModel
.populate({
path: 'users',
select: 'user role',
select: 'user role isNewJoiner',
...teamUserRoleFilter,
populate: {
path: 'user',
Expand All @@ -52,7 +52,18 @@ export default class GetTeamService implements GetTeamServiceInterface {
teamModel.lean();
}

return teamModel.exec();
return teamModel
.select('_id name')
.populate({
path: 'users',
select: 'user role isNewJoiner',
populate: {
path: 'user',
select: '_id firstName lastName email joinedAt'
}
})
.lean()
.exec();
}

async getTeamsOfUser(userId: string) {
Expand Down
2 changes: 1 addition & 1 deletion backend/src/modules/users/controller/users.controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export default class UsersController {
type: InternalServerErrorResponse
})
@Get()
getAllTeams() {
getAllUsers() {
return this.getUserApp.getAllUsers();
}
}
4 changes: 4 additions & 0 deletions frontend/src/api/teamService.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -14,3 +14,7 @@ export const getTeamsOfUser = (context?: GetServerSidePropsContext): Promise<Tea
export const createTeamRequest = (newTeam: CreateTeamDto): Promise<Team> => {
return fetchData(`/teams`, { method: 'POST', data: newTeam });
};

export const getTeamRequest = (id: string, context?: GetServerSidePropsContext): Promise<Team> => {
return fetchData(`/teams/${id}`, { context, serverSide: !!context });
};
19 changes: 18 additions & 1 deletion frontend/src/components/Primitives/Tooltip/styles.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,13 @@ import * as TooltipPrimitive from '@radix-ui/react-tooltip';

import { styled } from 'styles/stitches/stitches.config';

import {
slideDownAndFade,
slideLeftAndFade,
slideRightAndFade,
slideUpAndFade
} from '../../../animations/Slide';

const StyledContent = styled(TooltipPrimitive.Content, {
variants: {
color: {
Expand Down Expand Up @@ -39,7 +46,17 @@ const StyledContent = styled(TooltipPrimitive.Content, {
lineHeight: '$16',
textAlign: 'center',
border: 'none',
minWidth: '141px'
minWidth: '141px',
CatiaAntunes96 marked this conversation as resolved.
Show resolved Hide resolved
userSelect: 'none',
animationDuration: '400ms',
animationTimingFunction: 'cubic-bezier(0.16, 1, 0.3, 1)',
willChange: 'transform, opacity',
'&[data-state="delayed-open"]': {
'&[data-side="top"]': { animationName: slideDownAndFade },
'&[data-side="right"]': { animationName: slideLeftAndFade },
'&[data-side="bottom"]': { animationName: slideUpAndFade },
'&[data-side="left"]': { animationName: slideRightAndFade }
}
});

const StyledArrow = styled(TooltipPrimitive.Arrow, {
Expand Down
41 changes: 20 additions & 21 deletions frontend/src/components/Teams/CreateTeam/ListMembers/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -150,29 +150,28 @@ const ListMembers = ({ isOpen, setIsOpen }: Props) => {
Teams
</Text>
<ScrollableContent direction="column" justify="start" ref={scrollRef}>
<Flex
css={{ flex: '1 1', px: '$32', width: '80%' }}
direction="column"
gap={16}
>
<Flex css={{ flex: '1 1', px: '$32' }} direction="column" gap={16}>
{filteredList?.map((user) => (
<Flex key={user._id} align="center" justify="between">
<Checkbox
checked={user.isChecked}
disabled={user._id === session?.user.id}
handleChange={handleChecked}
id={user._id}
label={`${user.firstName} ${user.lastName}`}
size="16"
/>

<Text
color="primary300"
css={{ textAlign: 'left', width: '50%' }}
size="sm"
>
{user.email}
</Text>
<Flex css={{ width: '50%' }}>
<Checkbox
checked={user.isChecked}
disabled={user._id === session?.user.id}
handleChange={handleChecked}
id={user._id}
label={`${user.firstName} ${user.lastName}`}
size="16"
/>
</Flex>
<Flex css={{ width: '50%' }}>
<Text
color="primary300"
css={{ textAlign: 'left', width: '50%' }}
size="sm"
>
{user.email}
</Text>
</Flex>
</Flex>
))}
</Flex>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import Link from 'next/link';

import Icon from 'components/icons/Icon';
import Flex from 'components/Primitives/Flex';
import Text from 'components/Primitives/Text';
Expand All @@ -16,24 +14,22 @@ const BoardsInfo = ({ userSAdmin, teamAdmin, team }: BoardsInfoProps) => {
return (
<Flex css={{ ml: '$20', display: 'flex', alignItems: 'center' }}>
{(userSAdmin || teamAdmin) && (
<Link href="/boards/new">
<a style={{ textDecoration: 'none' }}>
<Flex css={{ alignItems: 'center' }}>
<Icon
name="plus"
css={{
width: '$16',
height: '$32',
marginRight: '$5'
}}
/>
<Text css={{ ml: '$8' }} size="sm" weight="medium">
{' '}
Create first team board
</Text>
</Flex>
</a>
</Link>
// <Link href="/boards/new">
<Flex css={{ alignItems: 'center' }}>
<Icon
name="plus"
css={{
width: '$16',
height: '$32',
marginRight: '$5'
}}
/>
<Text css={{ ml: '$8' }} size="sm" weight="medium">
{' '}
CatiaAntunes96 marked this conversation as resolved.
Show resolved Hide resolved
Create first team board
</Text>
</Flex>
// </Link>
)}
{!teamAdmin && (
<Text css={{ ml: '$14' }} size="sm" weight="medium">
Expand All @@ -46,11 +42,11 @@ const BoardsInfo = ({ userSAdmin, teamAdmin, team }: BoardsInfoProps) => {

return (
<Flex css={{ ml: '$20', display: 'flex', alignItems: 'center' }}>
<Link href="boards/">
<Text css={{ ml: '$14' }} size="sm" weight="medium">
{team.boardsCount} team boards
</Text>
</Link>
{/* <Link href="boards/"> */}
<Text css={{ ml: '$14' }} size="sm" weight="medium">
{team.boardsCount} team boards
</Text>
{/* </Link> */}
</Flex>
);
};
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/components/icons/Sprite.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -337,7 +337,7 @@ const Sprite = () => (
<path
clipRule="evenodd"
d="M6.36621 30.2746C14.1157 34.7636 25.2683 29.9783 28.9954 21.8554C32.7226 13.7325 30.8344 7.60134 23.0849 3.11237C15.3354 -1.37661 6.20395 -1.62868 2.47678 6.4942C-1.25039 14.6171 -1.3833 25.7857 6.36621 30.2746Z"
fill="#F3FD58"
fill="#CF97FF"
fillRule="evenodd"
/>
<path
Expand Down
23 changes: 18 additions & 5 deletions frontend/src/hooks/useTeam.tsx
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
import { useMutation, useQuery } from 'react-query';
import { useSetRecoilState } from 'recoil';

import { ToastStateEnum } from 'utils/enums/toast-types';
import { createTeamRequest, getAllTeams, getTeamsOfUser } from '../api/teamService';
import { toastState } from '../store/toast/atom/toast.atom';
import { createTeamRequest, getAllTeams, getTeamRequest, getTeamsOfUser } from '../api/teamService';
import UseTeamType from '../types/team/useTeam';
import useTeamUtils from './useTeamUtils';

interface AutoFetchProps {
autoFetchTeam: boolean;
}

const useTeam = ({ autoFetchTeam = false }: AutoFetchProps): UseTeamType => {
const setToastState = useSetRecoilState(toastState);
const { teamId, setToastState } = useTeamUtils();

const fetchAllTeams = useQuery(['allTeams'], () => getAllTeams(), {
enabled: autoFetchTeam,
refetchOnWindowFocus: false,
Expand All @@ -24,6 +24,18 @@ const useTeam = ({ autoFetchTeam = false }: AutoFetchProps): UseTeamType => {
}
});

const fetchTeam = useQuery(['team', teamId], () => getTeamRequest(teamId as string), {
CatiaAntunes96 marked this conversation as resolved.
Show resolved Hide resolved
enabled: autoFetchTeam,
refetchOnWindowFocus: false,
onError: () => {
setToastState({
open: true,
content: 'Error getting the team',
type: ToastStateEnum.ERROR
});
}
});

const fetchTeamsOfUser = useQuery(['teams'], () => getTeamsOfUser(), {
enabled: autoFetchTeam,
refetchOnWindowFocus: false,
Expand Down Expand Up @@ -56,7 +68,8 @@ const useTeam = ({ autoFetchTeam = false }: AutoFetchProps): UseTeamType => {
return {
fetchAllTeams,
fetchTeamsOfUser,
createTeam
createTeam,
fetchTeam
};
};

Expand Down
29 changes: 29 additions & 0 deletions frontend/src/hooks/useTeamUtils.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
import { useQueryClient } from 'react-query';
import { useRouter } from 'next/router';
import { useSession } from 'next-auth/react';
import { useSetRecoilState } from 'recoil';

import { toastState } from 'store/toast/atom/toast.atom';

const useTeamUtils = () => {
CatiaAntunes96 marked this conversation as resolved.
Show resolved Hide resolved
const router = useRouter();
const { data: session } = useSession({ required: false });

const queryClient = useQueryClient();

const userId = session?.user.id;

const setToastState = useSetRecoilState(toastState);

const { teamId } = router.query;

return {
userId,
teamId,
queryClient,
setToastState,
router
};
};

export default useTeamUtils;
1 change: 1 addition & 0 deletions frontend/src/types/team/useTeam.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,5 @@ export default interface UseTeamType {
createTeam: UseMutationResult<Team, unknown, CreateTeamDto, unknown>;
fetchAllTeams: UseQueryResult<Team[] | null, unknown>;
fetchTeamsOfUser: UseQueryResult<Team[] | null, unknown>;
fetchTeam: UseQueryResult<Team | null, unknown>;
}