Skip to content

feat: 조회하기 api 연결#60

Merged
dasosann merged 3 commits into
mainfrom
feat/matchinglist-api
Apr 30, 2026
Merged

feat: 조회하기 api 연결#60
dasosann merged 3 commits into
mainfrom
feat/matchinglist-api

Conversation

@dasosann

Copy link
Copy Markdown
Contributor

No description provided.

@qodo-code-review

Copy link
Copy Markdown
ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

이번 풀 리퀘스트에서는 구매 요청 상태를 조회하는 useRequestStatus 훅을 신설하고, 메인 페이지에서 구매 대기 상태에 따라 ChargeRequestWaiting 컴포넌트를 조건부로 노출하도록 구현했습니다. 또한 서버 사이드 프리페칭을 적용하고 매칭 리스트의 더미 데이터를 제거하는 등 전반적인 코드 정리도 함께 진행되었습니다. 리뷰에서는 컴포넌트에 흩어져 있는 비즈니스 로직(구매 대기 상태 판별)을 훅 내부로 캡슐화하여 가독성을 높일 것과, 데이터 플래트닝 과정에서 발생할 수 있는 런타임 에러를 방지하기 위해 옵셔널 체이닝을 보강할 것을 제안했습니다.

Comment thread app/main/_components/ScreenMainPage.tsx Outdated
Comment thread app/matching-list/_components/ScreenMatchingList.tsx Outdated
Comment thread hooks/useRequestStatus.ts
@dasosann
dasosann merged commit 5e6455f into main Apr 30, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant