fix: 공유페이지 등 동적 라우팅에서는 썸네일이 적용되지 않는 버그 수정#128
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded@KwonDeaGeun has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 5 seconds before requesting another review. ⌛ 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. 📒 Files selected for processing (1)
Walkthrough
Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related issues
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
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: 🔗 https://690458540424857aa71aec97-elvlkkcvvw.chromatic.com/ |
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
vercel.json(1 hunks)
🔇 Additional comments (1)
vercel.json (1)
2-7: 동적 라우팅 수정 사항을 검증하세요.SPA 리라이트 규칙이 추가되었으나, 썸네일(메타 태그) 문제가 완전히 해결되었는지 확인이 필요합니다. 소셜 미디어 공유 시 동적 페이지의 썸네일이 올바르게 표시되려면:
- 서버 사이드 렌더링(SSR) 또는 정적 생성(SSG)으로 동적 경로의 메타 태그 주입
- Open Graph API를 활용한 동적 메타 태그 생성
- 또는 Next.js ISR(Incremental Static Regeneration) 활용
이 설정 변경만으로는 메타 태그가 자동으로 생성되지 않습니다. 메타 태그 주입 로직(미들웨어, API 라우트 등)이 구현되어 있는지 확인하세요.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
vercel.json (1)
19-19: 선택사항: 패턴 문법 통일 제안Vercel 설정의 관례상 정적 자산 패턴을
/assets/(.*)에서/assets/:path*로 변경하면 더욱 일관성 있는 코드가 될 것입니다. 두 패턴 모두 동등하게 작동하지만,:path*문법이 Vercel의 현대적 설정 스타일과 더 잘 맞습니다.- "source": "/assets/(.*)", + "source": "/assets/:path*",
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
vercel.json(1 hunks)
🔇 Additional comments (3)
vercel.json (3)
2-7: ✅ 라우팅 규칙 수정으로 동적 라우팅 버그 해결파일시스템 핸들러를 제거하고 모든 경로를
/index.html로 리라이트하는 규칙을 추가하는 것은 SPA(Single Page Application) 클라이언트 라우팅을 올바르게 구성하는 표준 패턴입니다. 이 변경으로 동적 라우팅(공유페이지 등)에서 썸네일이 제대로 적용되지 않던 버그가 해결될 것으로 예상됩니다.
8-27: ✅ 이전 리뷰 피드백 반영된 캐시 정책HTML과 정적 자산에 대한 캐시 정책을 명확히 분리하는 좋은 개선입니다. 이전 리뷰에서 지적한 과도하게 제한적인 캐시 정책을 올바르게 해결했습니다:
/index.html은max-age=0으로 유지하여 항상 최신 버전을 제공/assets/(.*)는max-age=31536000, immutable으로 설정하여 Vite의 콘텐츠 기반 해시 파일명 전략을 최대한 활용
1-27: 검증 요청: 동적 라우팅에서 썸네일 표시 확인구성 변경이 올바른 것으로 보이지만, 실제 배포 후 동작을 확인해주시기 바랍니다:
- 공유페이지(또는 다른 동적 라우트)에서 썸네일이 제대로 표시되는지 확인
- 정적 자산(
/assets/파일들)이 여전히 올바르게 제공되는지 확인- 다른 라우트나 기능에 영향이 없는지 확인
Summary
해당 PR에 대한 작업 내용을 요약하여 작성해주세요.
Tasks
Summary by CodeRabbit
릴리스 노트
✏️ Tip: You can customize this high-level summary in your review settings.