Uh oh!
There was an error while loading. Please reload this page.
🐛 중앙대 슬러그가 university-web으로 rewrite되지 않던 문제 수정 - #624
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Warning Review limit reached
Next review available in:31 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. 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 |
Uh oh!
There was an error while loading. Please reload this page.
문제
https://www.solid-connection.com/university/chungang 이 404를 반환합니다.
원인
apps/web/next.config.mjs의 멀티존 rewrite 슬러그가 하드코딩되어 있었습니다.이 패턴이
/university/:homeUniversity요청을 university-web으로 넘길지 결정합니다.chungang이 목록에 없어 rewrite가 걸리지 않고, 해당 라우트가 없는apps/web으로 떨어져 404가 났습니다.#622(중앙대 추가)에서 놓친 두 번째 등록 지점입니다. 홈 대학은
apps/university-web의 상수와 이 rewrite 패턴 두 곳에 등록되어야 하는데, 전자만 반영했습니다.수정
슬러그 목록을 배열로 분리하고
chungang을 추가했습니다. 두 곳을 함께 갱신해야 한다는 점을 주석으로 명시했습니다./university/score/*,/university/application/*는apps/web소유이므로 와일드카드 대신 allowlist 구조를 유지했습니다. 실제로 생성되는 패턴을 확인했습니다.inha/kyunghee/chungangscore/application배포 순서 (중요)
이 PR 머지만으로는 사이트가 고쳐지지 않습니다.
release-university— main과 동일, 중앙대 포함 ✅ (university-web 쪽은 이미 준비 완료)release-web— main보다 15커밋 뒤처짐. rewrite를 소유한 앱이라 이 브랜치가 갱신되어야 반영됩니다.머지 후
Promote Main to Release Branchesworkflow를web타깃으로 실행해야/university/chungang이 열립니다.검증
pnpm --filter @solid-connect/web run lint:check— 통과 (university-zone-navigation 룰 포함)pnpm --filter @solid-connect/web run typecheck— 통과pnpm --filter @solid-connect/web run build— 통과후속 제안 (이 PR 범위 밖)
next.config.mjs가.mjs라 TS 상수를 import할 수 없어 수동 동기화가 남아 있습니다. 같은 사고가 반복되지 않으려면 슬러그 목록을 양쪽이 import할 수 있는 공용.mjs모듈로 빼거나, CI에서 두 목록의 일치를 검사하는 편이 안전합니다.또한
apps/web의HOME_UNIVERSITY_LIST에는 여전히 중앙대가 없습니다(홈 목록·sitemap).maxChoiceCount값이 확정되면 별도 PR로 반영하겠습니다.🤖 Generated with Claude Code