Open
Conversation
yeonjin719
requested changes
Feb 14, 2026
src/pages/main/HomePage/HomePage.tsx
Outdated
| select: (response) => response.result, | ||
| }) | ||
|
|
||
| const { data: reminders } = useCustomQuery(['reminders'], fetchReminders, { |
Collaborator
There was a problem hiding this comment.
훅을 분리하고 query key는 query key factory로 전환해주세용
There was a problem hiding this comment.
Pull request overview
이 PR은 홈 페이지에 오늘의 브리핑과 리마인더 API를 연결하는 작업입니다. src/shared 레이어에 타입과 API 함수를 추가하고, HomePage 컴포넌트에서 이를 소비하여 실제 데이터를 렌더링하도록 구현되었습니다.
Changes:
- 홈 관련 타입 정의 추가 (
BriefingResponse,ReminderResponse등) - 브리핑 및 리마인더 조회 API 함수 구현
HomePage컴포넌트에서 API 연동 및 데이터 기반 UI 렌더링
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 9 comments.
| File | Description |
|---|---|
src/shared/types/home/home.ts |
브리핑 및 리마인더 관련 TypeScript 타입 정의 추가 |
src/shared/api/home/home.ts |
브리핑 및 리마인더 조회 API 함수 구현 |
src/pages/main/HomePage/HomePage.tsx |
하드코딩된 데이터를 API 호출 결과로 대체하고 로딩/상태 처리 추가 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 관련 이슈
🧩 작업 요약 (TL;DR)
🔄 변경 유형
해당되는 항목에 체크해주세요.
📌 주요 변경 사항
N/A
🖼️ 스크린샷 / 영상 (선택)
N/A
🧠 리뷰 요청 포인트
🚧 미완 / 후속 작업
💬 기타 참고 사항
N/A
@copilot 이 PR을 아래 기준으로 검토해주세요:
.github/instructions/capstone.instructions.md 파일을 지침으로 삼으세요.
폴더/파일 위치가 프로젝트 구조 규칙과 일치하는지
컴포넌트가 단일 책임 원칙(SRP)을 지키는지
import 방향이 올바른지 (shared → features 역방향 없음)
명명/케이스가 일관적인지 (PascalCase vs camelCase)
배럴(index.ts) 사용이 현 패턴을 따르는지
응답은 한국어로, 발견된 위반 항목과 추천 구조를 포함해주세요.
리뷰를 달아주세요