Skip to content

fix: 프로덕션 색인 대상 정리 #599

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
manNomi merged 1 commit into from
Jul 6, 2026
Merged

fix: 프로덕션 색인 대상 정리 #599

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 0 additions & 6 deletions apps/university-web/src/app/sitemap.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,6 @@ export default function sitemap(): MetadataRoute.Sitemap {
changeFrequency: "weekly",
priority: 0.8,
},
{
url: `${siteUrl}/university/search`,
lastModified,
changeFrequency: "weekly",
priority: 0.6,
},
...HOME_UNIVERSITY_SLUGS.map((slug) => ({
url: `${siteUrl}/university/${slug}`,
lastModified,
Expand Down
18 changes: 14 additions & 4 deletions apps/web/next.config.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,20 @@ const nextConfig = {
typescript: {
ignoreBuildErrors: true,
},
async redirects() {
return [
{
source: `/university/list/:homeUniversity${universitySlugPattern}`,
destination: "/university/:homeUniversity",
permanent: true,
},
{
source: `/university/list/:homeUniversity${universitySlugPattern}/:path*`,
destination: "/university/:homeUniversity/:path*",
permanent: true,
},
];
},
async rewrites() {
if (!universityWebDomain) {
return [];
Expand All @@ -96,10 +110,6 @@ const nextConfig = {
source: "/university/search",
destination: `${universityWebDomain}/university/search`,
},
{
source: `/university/list/:homeUniversity${universitySlugPattern}`,
destination: `${universityWebDomain}/university/:homeUniversity`,
},
{
source: `/university/:homeUniversity${universitySlugPattern}`,
destination: `${universityWebDomain}/university/:homeUniversity`,
Expand Down
1 change: 0 additions & 1 deletion apps/web/src/app/sitemap.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ const toSitemapEntry = (
const getStaticRoutes = (): MetadataRoute.Sitemap => [
toSitemapEntry("/", "daily", 1),
toSitemapEntry("/university", "weekly", 0.85),
toSitemapEntry("/university/search", "weekly", 0.6),
toSitemapEntry("/terms", "yearly", 0.35),
...HOME_UNIVERSITY_LIST.map((university) => toSitemapEntry(`/university/${university.slug}`, "weekly", 0.8)),
];
Expand Down
2 changes: 1 addition & 1 deletion docs/university-multizone-deployment.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
- `/university/:homeUniversity/:id`
- `/university-static/*`

`/university/list/:homeUniversity`는 legacy URL로만 취급한다. child route는 유지하지 않고, main rewrite에서 `/university/:homeUniversity` SSG 페이지로 보낸다. 이렇게 하면 오래된 외부 링크는 살리면서 SSG catalog 안에 client-fetch 목록 페이지가 섞이지 않는다.
`/university/list/:homeUniversity`는 legacy URL로만 취급한다. child route는 유지하지 않고, main redirect에서 `/university/:homeUniversity` SSG 페이지로 보낸다. 이렇게 하면 오래된 외부 링크는 canonical URL로 정리하면서 SSG catalog 안에 client-fetch 목록 페이지가 섞이지 않는다.

@coderabbitai coderabbitai Bot Jul 6, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n== Files ==\n'
git ls-files | rg '^(docs/university-multizone-deployment\.md|apps/web/next\.config\.mjs)$'

printf '\n== Doc snippet ==\n'
cat -n docs/university-multizone-deployment.md | sed -n '28,40p'

printf '\n== next.config search ==\n'
rg -n '/university/list|redirect|pathname|source|destination' apps/web/next.config.mjs

printf '\n== next.config snippet ==\n'
cat -n apps/web/next.config.mjs | sed -n '70,95p'

Repository: solid-connection/solid-connect-web

Length of output: 3138

1. 문구를 하위 경로 리다이렉트까지 맞춰주세요.
apps/web/next.config.mjs에는 /university/list/:homeUniversity/:path* -> /university/:homeUniversity/:path* 규칙도 있어서, child route가 그대로 canonical 하위 경로로 이어집니다. 지금의 “child route는 유지하지 않는다”는 실제 동작과 어긋나니, 이 부분까지 반영해 문구를 다듬어 주세요.

🤖 Prompt for 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.

In `@docs/university-multizone-deployment.md` at line 34, The current wording
around /university/list/:homeUniversity only describes the main redirect, but
the actual behavior also includes child-route forwarding via the
legacy-to-canonical redirect rules. Update the documentation text to explicitly
mention that /university/list/:homeUniversity/:path* is redirected to
/university/:homeUniversity/:path*, so the description matches the behavior
defined by the redirect logic and the Next.js config.


## Vercel 프로젝트 설정

Expand Down
Loading