[Bugfix/#184] 트래픽 차트 스켈레톤 고착 / 사이드바 가로 스크롤 수정 - #185
Conversation
- 사이드바 열림 애니메이션 중 overflow-x-hidden 추가로 가로 스크롤바 노출 방지 - fetchEventSource URL에 VITE_API_BASE_URL 적용하여 Vercel 배포 환경에서 SSE 연결 실패 수정
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, 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 have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough대시보드 플랫폼 뷰의 트래픽 차트 플레이스홀더를 추가하고, 사이드바 오버플로우 동작을 수정했으며, 설정 메뉴를 푸터로 이동하고, API 스트림 요청 URL 생성 방식을 정규화했습니다. Changes플레이스홀더 및 네비게이션 개선
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
📚 Storybook 배포 완료
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
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/hooks/dashboard/useClickStream.ts`:
- Around line 37-42: onopen 핸들러에서 현재 response.ok만 검사하는 대신
response.headers.get('content-type')가 'text/event-stream'을 포함하는지 명시적으로 검사하여 유효한
SSE 응답인지 확인하세요; 만약 content-type 검사에 실패하면 onopen에서 예외를 던지거나 retry 흐름으로 넘겨
retryCountRef.current/reset 로직이 작동하도록 하고, 기존에 사용 중인 retryCountRef, setIsError 로
상태를 적절히 업데이트하도록 수정하세요 (참조: onopen 콜백, retryCountRef.current, setIsError).
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: fb7b190c-ef28-40f4-89f9-08718412db14
📒 Files selected for processing (5)
src/components/dashboard/platform/AllPlatformView.tsxsrc/components/dashboard/platform/SinglePlatformView.tsxsrc/components/sidebar/Sidebar.tsxsrc/constants/sidebarNav.tssrc/hooks/dashboard/useClickStream.ts
💤 Files with no reviewable changes (1)
- src/constants/sidebarNav.ts
|
P4: 확인했습니다!! |
2 participants
🚨 관련 이슈
#184
✨ 변경사항
✏️ 작업 내용
overflow-x-hidden추가로 가로 스크롤바 노출 방지<원인>
사이드바 토글 시
isCollapsed상태가 즉시 전환되어 텍스트 span이whitespace-nowrap전체 폭으로 즉시 렌더링되지만, Framer Motion의 width애니메이션(88px → 256px)은 ~300ms에 걸쳐 진행됩니다.
이 타이밍 불일치로 내부 콘텐츠가 컨테이너를 초과하고,
overflow-y: auto가CSS 스펙상
overflow-x도auto로 강제 전환하면서 가로 스크롤바가 노출됐습니다.mainNav에 중복 등록된 설정 아이템 제거,footerNav에만 유지fetchEventSourceURL에VITE_API_BASE_URL적용하여 Vercel 배포 환경에서 catch-all rewrite에 걸리던 문제 해결<원인>
fetchEventSource는 axios를 거치지 않아 axiosInstance의 baseURL
설정이 적용되지 않습니다. 기존 상대 경로(/api/...)는 개발 환경에서는
Vite proxy가 처리해줬지만, Vercel에서는 vercel.json의 catch-all rewrite
("source": "/(.*)")에 걸려 index.html을 응답받았습니다.
SSE 이벤트를 수신하지 못해 data가 null로 유지되고 스켈레톤만 표시됐습니다.
😅 미완성 작업
N/A
📢 논의 사항 및 참고 사항
사이드바 내부 설정 아이콘 2개 있었는데 확인해보니
chore: pageHeader삭제, profileSection내 조직 직책 직관성 향상 커밋(55692a7)해당 커밋에서 추가된 듯하여 삭제했습니다...Summary by CodeRabbit
릴리스 노트
새로운 기능
스타일 개선
네비게이션 변경