Skip to content

Comments

fix: 대학 리스트 상세 링크를 홈대학 slug 기반으로 수정#454

Merged
manNomi merged 1 commit intomainfrom
fix/university-list-detail-link-slug
Feb 20, 2026
Merged

fix: 대학 리스트 상세 링크를 홈대학 slug 기반으로 수정#454
manNomi merged 1 commit intomainfrom
fix/university-list-detail-link-slug

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 20, 2026

Summary

  • 대학 카드 상세 링크를 homeUniversityName 기반 slug 매핑으로 생성하도록 수정했습니다.
  • 상세 링크가 \/university/{slug}/{id} 형태(예: /university/inha/1899)로 이동하도록 보정했습니다.
  • 홈 인기 대학 카드에서도 동일한 slug 기반 상세 링크 규칙을 적용했습니다.

Changed Files

  • apps/web/src/constants/university.ts
  • apps/web/src/components/ui/UniverSityCard/index.tsx
  • apps/web/src/app/(home)/_ui/PopularUniversitySection/_ui/PopularUniversityCard.tsx

Verification

  • pnpm --filter @solid-connect/web run ci:check
  • pnpm --filter @solid-connect/web run build

@vercel
Copy link

vercel bot commented Feb 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Feb 20, 2026 2:29pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Feb 20, 2026 2:29pm

@github-actions github-actions bot added the web label Feb 20, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

대학교 네비게이션 로직을 개선하는 세 가지 변경사항이 포함되었습니다.

  1. university.ts에 헬퍼 함수 추가 — getHomeUniversitySlugByName 함수가 새로 추가되어 대학교 이름을 정규화하고 HOME_UNIVERSITY_TO_SLUG_MAP에서 해당하는 슬러그를 조회합니다.

  2. PopularUniversityCard.tsx 업데이트 — 새로운 헬퍼 함수를 사용하여 universityDetailHref를 계산하고, 슬러그가 존재하면 /university/{slug}/{id} 형식으로, 그렇지 않으면 /university로 라우팅하도록 Link href를 변경합니다.

  3. UniverSityCard/index.tsx 업데이트 — 동일한 헬퍼 함수를 활용하여 동적 라우팅 로직을 구현하며, linkPrefix가 기본값이 아닌 경우 {linkPrefix}/{id} 경로를 사용하는 조건부 처리를 추가합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경사항을 명확하게 반영하고 있습니다. 홈대학 slug 기반 상세 링크 수정이라는 핵심 내용을 간결하게 표현했습니다.
Description check ✅ Passed PR 설명이 작업 내용을 명확하게 기술하고 있으며, 변경된 파일과 검증 단계를 포함하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/university-list-detail-link-slug

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@manNomi manNomi merged commit 26e0502 into main Feb 20, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant