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

[탈퇴, 팀관리] 탈퇴회원 상세정보, 팀관리 구현 #120

Merged
merged 8 commits into from
Mar 25, 2024

Conversation

MinGu-Jeong
Copy link
Collaborator

[#114 ] request

  • 회원을 삭제할 때 탈퇴사유를 작성할 수 있으며, 수정하고 조회할 수 있습니다.
    • 회원정보에서 탈퇴사유는 NORMAL권한을 가진 아이디에서 조회가능하며, ADMIN이나 MANAGER권한을 가진 아이디는 회원삭제 화면에서 탈퇴사유 조회가 가능합니다.
  • 팀 관리 페이지를 구현했습니다. 현재 아래 기능들이 구현되었습니다.
    • 팀 추가
    • 팀 수정
    • 팀장, 팀원 조회

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Did you merge recent main branch?

Screenshot

스크린샷 2024-03-25 오후 3 05 59 스크린샷 2024-03-25 오후 3 06 18 스크린샷 2024-03-25 오후 3 05 40

Precautions (main files for this PR ...)

팀원 관리기능은 스프린트 종료시점까지 PR올리기가 힘들 것 같아, 추후 PR을 따로 올리도록 하겠습니다.

@MinGu-Jeong MinGu-Jeong added the enhancement New feature or request label Mar 25, 2024
@MinGu-Jeong MinGu-Jeong self-assigned this Mar 25, 2024
Copy link
Contributor

@chaeseungyun chaeseungyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드가 많아서 리뷰하기 힘들군요..

고생하셨습니다

// const [teamMemberModalOpen, setTeamMemberModalOpen] = useState(false);
const [teamInfoModalOpen, setTeamInfoModalOpen] = useState(false);
const [selectedTeamId, setSelectedTeamId] = useState<number | null>(null);
const { data: getMe } = useGetMe();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

로그인 상태를 전역적으로 관리하고 있기 때문에 GetMe를 또 호출하지 않아도 됩니다
useLoginState에서 가져다 사용하는 것을 추천합니다

<div css={S.container}>
<div css={S.contentContainer}>
<Grid container spacing={2}>
{teams?.map((team: Team) => (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useSuspenseQuery를 사용하셨는데 teams가 undefined로도 추정되나요?

@MinGu-Jeong MinGu-Jeong merged commit eebb88a into main Mar 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[팀 관리] 팀 관리 페이지 구현
2 participants