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

대시보드 내가 제출한 디스커션 구현 (issue #482) #484

Open
wants to merge 10 commits into
base: dev
Choose a base branch
from

Conversation

brgndyy
Copy link
Contributor

@brgndyy brgndyy commented Sep 20, 2024

구현 요약

대시보드 내가 제출한 디스커션 구현 (isseu #440 )과 파일이 커밋 내용 동일합니다!

연관 이슈

참고

코드 리뷰에 RCA 룰을 적용할 시 참고해주세요.

헤더 설명
R (Request Changes) 적극적으로 반영을 고려해주세요
C (Comment) 웬만하면 반영해주세요
A (Approve) 반영해도 좋고, 넘어가도 좋습니다. 사소한 의견입니다.

@brgndyy brgndyy added 🎨 프론트엔드 프론트엔드 관련 이슈 ⚒️ 기능 작업해야하는 기능 labels Sep 20, 2024
@brgndyy brgndyy self-assigned this Sep 20, 2024
Copy link
Contributor

@chosim-dvlpr chosim-dvlpr left a comment

Choose a reason for hiding this comment

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

안녕하세요 버건디~ 앞서 열려있었던 PR 확인했습니다 ㅎㅎ
dashboardKeys 속성 관련 코멘트도 짧게 남겨놓았어요! 구조가 복잡한 부분 구현하시느라 고생많으셨습니다!!👍🏻


export const dashboardKeys = {
default: ['dashboard'],
discussion: () => [...dashboardKeys.default, 'discussions'],
Copy link
Contributor

Choose a reason for hiding this comment

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

[A]
discussion List를 받아오는 쿼리의 key값이라 속성 값 discussion을 복수형인 discussions로 하면 좋을 것 같은데 어떠실까요?!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚒️ 기능 작업해야하는 기능 🎨 프론트엔드 프론트엔드 관련 이슈
Projects
Status: ✏️ TODO
Development

Successfully merging this pull request may close these issues.

대시보드 기능 구현
2 participants