Skip to content

Conversation

@l-lyun
Copy link
Contributor

@l-lyun l-lyun commented Aug 19, 2025

#️⃣연관된 이슈

📝작업 내용

리뷰 관리 페이지 내 댓글을 기준으로 분리하였습니다.

  • review get 관련 훅 쿼리키 변경
  • useUnRepliedReviewList 훅 추가

스크린샷 (선택)

image image

💬리뷰 요구사항(선택)

@l-lyun l-lyun requested a review from 99mini August 19, 2025 13:26
@l-lyun l-lyun self-assigned this Aug 19, 2025
@l-lyun l-lyun added the feature label Aug 19, 2025
Comment on lines +25 to +27
const {onRefresh, refreshing} = usePullDownRefresh(async () => {
await queryClient.invalidateQueries({
queryKey: ['marketList', 'review'],
Copy link
Member

Choose a reason for hiding this comment

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

무효화할때마다 쿼리키가 무엇인지 api 쿼리를 확인해야되서 약간 불편할 수도 있겠다는 생각이 드네요..

필요하다면 유틸로 묶어서 queryclient를 키를 받고, 쿼리키는 타입추론이 될수있도록 나중에 해보죠

Copy link
Contributor Author

@l-lyun l-lyun Aug 20, 2025

Choose a reason for hiding this comment

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

저도 해당 부분 쿼리키 훅 만지면서 고민했는데, 프로젝트 사이즈가 더 커지면 형님 말씀해주신 코드 내에서 방법이나,

쿼리키에 대해 문서화를 확실히 해야겠다는 생각이 들었습니다. 코드적으로 어떻게 풀 수 있지?
하고 넘어갔는데 조언 감사합니다~!

@l-lyun l-lyun merged commit 6a3d790 into main Aug 20, 2025
1 check passed
@l-lyun l-lyun deleted the feature/review-no-reply-tabbar branch August 20, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants