Skip to content

Comments

Refactor/103/code review#110

Merged
cksrlcks merged 4 commits intomainfrom
refactor/103/code-review
Feb 12, 2025
Merged

Refactor/103/code review#110
cksrlcks merged 4 commits intomainfrom
refactor/103/code-review

Conversation

@cksrlcks
Copy link
Collaborator

@cksrlcks cksrlcks commented Feb 11, 2025

❓이슈

✍️ Description

코드리뷰 받은 내용 개선했습니다.
통신관련 주요 함수들이 개선되어 모두가 리뷰를 해야할 것 같습니다.

  • useQuery 개선 (실수 방지를 위해 파라미터를 통채로 넘기도록 개선, 단일값을 받는경우에는 최대한 작성되어있는 기존의 타입을 이용하도록 개선)
  • useMutation 개선 (기존에 하나의 mutation으로 export를 각각 분리)
  • api 서비스 함수 개선 (타입 강화 및 네이밍 수정, 주석 추가)
  • safeParse 부분 공통 함수로 분리
  • types 개선 (공용 타입 분리 및 네이밍 정리)
  • 불필요한 suspense 제거
  • 대시보드 정보 수정페이지에서 ssr 데이터 다시 받도록 수정 (router.refresh() 이용)

✅ Checklist

PR

  • Branch Convention 확인

    feat/* 기능 구현, fix/* 버그 수정, refactor/* 개선

  • Base Branch 확인
  • 적절한 Label 지정
  • Assignee 및 Reviewer 지정

Test

  • 로컬 작동 확인

Additional Notes

  • (없음)

@cksrlcks cksrlcks self-assigned this Feb 11, 2025
@cksrlcks cksrlcks linked an issue Feb 11, 2025 that may be closed by this pull request
5 tasks
@vercel
Copy link

vercel bot commented Feb 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 11, 2025 5:50pm

Copy link
Owner

@ToKyun02 ToKyun02 left a comment

Choose a reason for hiding this comment

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

멘토링 내용 반영한 것 확인했습니다!

safeParse를 util로 묶은 신 것 좋다고 생각합니다!

@cksrlcks cksrlcks merged commit f5465a2 into main Feb 12, 2025
3 checks passed
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.

Bug : 대시보드 정보수정시 타이틀 안바뀌는 현상 수정 Refactor : 코드리뷰 사항 반영 (api, react-query, type 관련)

2 participants