Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Walkthrough리마인드 API를 v1 인라인 쿼리 문자열 방식에서 v2 파라미터 객체 방식으로 마이그레이션하고, 페이지네이션 종료 조건을 배열 길이 확인에서 hasNext 플래그 확인으로 변경했습니다. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Important Action Needed: IP Allowlist UpdateIf your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:
Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Storybook chromatic 배포 확인: |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
apps/client/src/pages/remind/apis/queries.ts (1)
10-15:⚠️ Potential issue | 🟠 Major
lastPage응답 타입 정의 필요
ArticleListResponse인터페이스에는hasNext속성이 정의되어 있지 않으나, 코드에서lastPage.hasNext에 접근하고 있습니다. 또한getRemindArticles함수에 반환 타입 주석이 없어 타입 안전성이 보장되지 않습니다.실제 API 응답에는
hasNext필드가 포함되어 있으므로, 타입 정의를 수정하여 API 응답 구조와 일치시키거나 명시적인 반환 타입을 추가해야 합니다.💡 타입 정의 예시
// apps/client/src/pages/remind/types/api.ts export interface ArticleListResponse { readArticleCount: number; unreadArticleCount: number; articles: ArticleWithCategory[]; hasNext: boolean; // 추가 필요 } // apps/client/src/pages/remind/apis/axios.ts export const getRemindArticles = async ( nowDate: string, readStatus: boolean, page: number, size: number ): Promise<ArticleListResponse> => { // 반환 타입 명시 // ... };
constantly-dev
left a comment
There was a problem hiding this comment.
굿입니다~ 👍
혹시 해당 API 변동 이유가 뭐였나요??
📌 Related Issues
📄 Tasks
⭐ PR Point (To Reviewer)
같은 토큰으로 로컬과 프로드 환경에서 테스트해봤습니다.
북마크에서 읽음이어도 리마인드에 새로 뜨면 안읽음으로 뜨는거 확인했습니다.
📷 Screenshot
로컬 테스트(변경 o)

프로드 테스트(변경 x)

Summary by CodeRabbit
Release Notes