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

feat(member): 활동 그룹 관리 및 과제 제출 조회 및 피드백 생성 #201

Merged
merged 31 commits into from
Aug 17, 2024

Conversation

Jeong-Ag
Copy link
Contributor

Summary

#39

활동 그룹 관리를 위한 컴포넌트를 생성하고 과제 조회와 피드백 기능을 구현합니다.

Tasks

  • 활동 그룹 공지사항, 주차별 활동, 과제 관리 기능
  • 과제 제출 조회 및 미제출 명단 조회
  • 과제 피드백 기능

ETC

  • 부모 게시판 하위의 게시판 리스트를 나타내는 경우 하위 게시판이 수정되거나 추가되어도 리스트에 즉시 반영되지 않고 새로고침 해야 변경된 내용이 반영됩니다 (ex 관리 페이지 내에서 주차별 활동에 과제를 생성한 경우 바로 조회되지 않고 새로고침 해야 나타남).
  • query key를 DETAIL, BOARD 등으로 나누어 사용하는 부분이 원인으로 예상됩니다. 해당 부분 관련하여 고민한 후 리팩토링 하겠습니다.
  • 활동 그룹 정보 관리 기능은 백엔드 api 수정 완료되면 연동하겠습니다.

Screenshot

스크린샷 2024-08-14 오후 10 04 47 스크린샷 2024-08-14 오후 10 30 18 스크린샷 2024-08-14 오후 6 48 14 스크린샷 2024-08-14 오후 6 48 22 스크린샷 2024-08-14 오후 6 43 15 스크린샷 2024-08-14 오후 6 43 18 스크린샷 2024-08-14 오후 6 43 07

@Jeong-Ag Jeong-Ag added ✨ Feature 새로운 기능 명세 및 개발 🔨 Refactor 코드 수정 및 최적화 🏠 Member member 프로젝트 관련 labels Aug 14, 2024
@Jeong-Ag Jeong-Ag requested review from gwansikk and SWARVY August 14, 2024 15:33
@Jeong-Ag Jeong-Ag self-assigned this Aug 14, 2024
Copy link

changeset-bot bot commented Aug 14, 2024

⚠️ No Changeset found

Latest commit: 6b90923

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@gwansikk gwansikk changed the title 활동 그룹 관리 및 과제 제출 조회 및 피드백 생성 feat(member): 활동 그룹 관리 및 과제 제출 조회 및 피드백 생성 Aug 15, 2024
Copy link
Contributor

@SWARVY SWARVY left a comment

Choose a reason for hiding this comment

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

수고 많으셨습니다잇

@Jeong-Ag Jeong-Ag requested review from SWARVY and gwansikk August 15, 2024 12:37
size = 20,
}: useActivityGroupBoardByParentProps) {
return useSuspenseQuery({
queryKey: [...ACTIVITY_QUERY_KEY.BOARDS(), 'by-parent', parentId],
Copy link
Member

@gwansikk gwansikk Aug 15, 2024

Choose a reason for hiding this comment

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

해당 프로젝트에서는 QueryKey를 Query Key Factory를 사용하여 관리하고 있어요, 해당 코드에서 사용하는 것 처럼 by-parentparentId를 선언하여 사용하면, 이후 QueryKey를 관리하기 어려워져요. ACTIVITY_QUERY_KEY에 QueryKey를 선언적으로 사용할 수 있도록 개선해보는건 어떨까요

ref: https://tkdodo.eu/blog/effective-react-query-keys#use-query-key-factories

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다 남겨주신 글 참고해서 읽어보고 추가적으로 선언하여 개선하겠습니다~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

현재 별도로 분리하지 않고 BOARD의 params로 설정하였는데 따로 BOARD_CATEGORY나 BOARD_PARENT로 별도 선언하는 방법과 비교하면 어떤 것이 더 관리하기 좋은 방법일지 여쭤보고 싶습니다. 쿼리 키가 많아질 것을 걱정하여 위와 같은 방법으로 사용했는데 별도 선언하는 것이 더 직관적이고 관리하기 쉬울까요?

Copy link
Member

@gwansikk gwansikk Aug 17, 2024

Choose a reason for hiding this comment

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

제 생각에는 별도로 분리하는게 관리성이 좋다 입니다, 쿼리키는 다른 파일에서도 사용할 수 있는데요(쿼리키를 비활성화) 따로 분리하여 관리할 경우 직관적이고 명시적으로 사용할 수 있다고 생각해요

@Jeong-Ag Jeong-Ag requested a review from gwansikk August 17, 2024 10:43
Copy link
Contributor

@SWARVY SWARVY left a comment

Choose a reason for hiding this comment

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

고민한 흔적이 많이 보이네요! 리뷰 내용 확인하시고 필요한 부분은 수정해주세요 :)

수고 많으셨습니다


const feedback = myAssignment?.[0].feedbacks?.[0];
const isLeader = groupDetail?.groupMembers.some(
Copy link
Contributor

@SWARVY SWARVY Aug 17, 2024

Choose a reason for hiding this comment

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

배열 내장함수를 적극적으로 사용하는거 보기 좋아요 bb

apps/member/src/api/activity.ts Outdated Show resolved Hide resolved
@Jeong-Ag Jeong-Ag requested a review from SWARVY August 17, 2024 12:06
@SWARVY SWARVY merged commit 5259e47 into main Aug 17, 2024
4 checks passed
@gwansikk gwansikk deleted the feature/#39 branch August 17, 2024 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 새로운 기능 명세 및 개발 🏠 Member member 프로젝트 관련 🔨 Refactor 코드 수정 및 최적화
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants