Skip to content

Comments

fix: 메인 인기 파견학교 상세 링크 홈대학 매핑 보강#458

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

fix: 메인 인기 파견학교 상세 링크 홈대학 매핑 보강#458
manNomi merged 1 commit intomainfrom
fix/home-popular-university-detail-link

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 20, 2026

Summary

  • 메인 페이지 추천 대학 응답에 homeUniversityName이 비어 있을 때, 권역별 전체 대학 데이터(ALL)에서 같은 id의 홈대학명을 보강하도록 변경했습니다.
  • 보강된 데이터를 PopularUniversitySection에 전달해 인기 파견학교 카드가 /university/{homeUniversitySlug}/{id} 상세 경로로 이동할 수 있게 했습니다.
  • 기존 데이터가 이미 완전한 경우에는 기존 값을 그대로 사용하도록 안전하게 병합했습니다.

Verification

  • pnpm --filter @solid-connect/web run typecheck
  • pnpm --filter @solid-connect/web run build
  • pre-commit/pre-push 훅의 ci:check 및 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 4:29pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Feb 20, 2026 4: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. RegionEnumExtend 임포트 추가

    • 필요한 타입 정보를 가져왔습니다.
  2. homeUniversityName 데이터 보강 로직

    • 모든 대학교 데이터에서 ID와 homeUniversityName을 매핑한 맵을 구성합니다.
    • 추천 대학교 목록 중 빠진 homeUniversityName 값들을 맵에서 채워 넣습니다.
  3. 컴포넌트 렌더링 연결

    • PopularUniversitySection에 기존 데이터 대신 보강된 데이터(resolvedRecommendedUniversities)를 전달합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR 설명에서 '관련 이슈' 섹션이 누락되어 있습니다. 나머지 작업 내용과 검증 정보는 명확하게 작성되었지만, 필수 템플릿 섹션이 완전하지 않습니다. 연관된 이슈 번호를 '## 관련 이슈' 섹션에 추가하여 PR의 추적성을 높여주세요. 예: resolves: #이슈번호
✅ Passed checks (2 passed)
Check name Status Explanation
Title 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/home-popular-university-detail-link

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 066c86a 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