feat(IN-238): 인플루언서 검색 상세 페이지 헤더(채널 프로필) 북마크, 해시태그 추가#135
Conversation
|
@juuhye is attempting to deploy a commit to the kimyunhyeong's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Warning Review limit reached
More reviews will be available in 1 minute and 3 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughPR 135는 인플루언서 상세 페이지에 북마크 기능을 통합합니다. Changes인플루언서 북마크 기능 통합
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/widgets/influencerDetail/influencerBaseInfo/ui/InfluencerBaseInfo.tsx`:
- Around line 84-86: The bookmark toggle currently calls toggleBookmark with
data.channelId which can differ from the page route and cause the wrong channel
to be updated; change the onToggle handler to use the route's channelId (the
page-level identifier you obtain from useParams or props — e.g., the route
variable named channelId) instead of data.channelId so
toggleBookmark(routeChannelId, bookmarked) is always invoked; update the
onToggle reference in the InfluencerBaseInfo component (where onToggle and
toggleBookmark are used) to pass that route channelId.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: bbbe2c74-4da3-4e24-a4f6-7e0f4b1acb13
📒 Files selected for processing (4)
src/entities/influencerDetail/mock/mockInfluencerDetail.tssrc/entities/influencerDetail/model/types.tssrc/features/influencer/index.tssrc/widgets/influencerDetail/influencerBaseInfo/ui/InfluencerBaseInfo.tsx
📌 작업 개요
🗂 작업 유형
✏️ 작업 내용
✅ 셀프 체크리스트
💬 리뷰어에게
Summary by CodeRabbit
릴리스 노트