Skip to content

fix: 홈대학 짧은 명칭 매칭 및 대학 미존재 폴백 처리#452

Merged
manNomi merged 1 commit intomainfrom
fix/home-university-shortname-fallback
Feb 19, 2026
Merged

fix: 홈대학 짧은 명칭 매칭 및 대학 미존재 폴백 처리#452
manNomi merged 1 commit intomainfrom
fix/home-university-shortname-fallback

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 19, 2026

Summary

  • 홈대학 필터링 로직에 정식명/짧은명 정규화 매칭을 추가해 인하대학교인하대 모두 동일하게 처리하도록 수정했습니다.
  • 대학 상세에서 404(대학 미존재) 응답도 notFound 대신 폴백 UI를 노출하도록 변경했습니다.
  • 대학 상세 폴백 컴포넌트에 제목/설명 커스터마이징 props를 추가해 오류 상황별 안내 문구를 분기했습니다.

Changed Files

  • apps/web/src/constants/university.ts
  • apps/web/src/app/university/[homeUniversity]/page.tsx
  • apps/web/src/app/university/[homeUniversity]/[id]/page.tsx
  • apps/web/src/app/university/[homeUniversity]/[id]/_ui/UniversityDetailPreparingFallback.tsx
  • apps/web/src/apis/universities/getSearchText.ts
  • apps/web/src/apis/universities/getSearchFilter.ts

Verification

  • lsp_diagnostics (modified files): no errors
  • pnpm --filter @solid-connect/web run build
  • pnpm --filter @solid-connect/web run ci:check

@vercel
Copy link

vercel bot commented Feb 19, 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 19, 2026 2:01pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Feb 19, 2026 2:01pm

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

coderabbitai bot commented Feb 19, 2026

Caution

Review failed

The pull request is closed.

워크스루

이번 PR은 대학교 이름 매칭 로직을 개선하는 변경사항입니다. 다음과 같은 수정이 포함되어 있습니다:

  1. 새로운 유틸리티 함수 추가

    • normalizeHomeUniversityName: 문자열을 정규화하여 기존 HOME_UNIVERSITY_LIST와 매칭
    • isMatchedHomeUniversityName: 두 문자열을 정규화한 후 동일한 대학으로 매칭되는지 확인
  2. 기존 비교 로직 개선

    • 6개 파일에서 엄격한 동등성 검사(===)를 isMatchedHomeUniversityName 호출로 교체
  3. 컴포넌트 인터페이스 확장

    • UniversityDetailPreparingFallback에 선택적 props(title, description) 추가
  4. 에러 처리 흐름 변경

    • 404 에러 시 직접 notFound() 호출 대신 UI 폴백을 조건부 메시징과 함께 사용

예상 코드 리뷰 난이도

🎯 3 (Moderate) | ⏱️ ~20 분

제안하는 리뷰어

  • wibaek
  • enunsnv
  • khwww
✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/home-university-shortname-fallback

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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 d6a82e8 into main Feb 19, 2026
9 of 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

Comments