Skip to content

Comments

fix: 대학 정보 표기의 CURRENT_TERM env 의존 제거#456

Merged
manNomi merged 2 commits intomainfrom
fix/university-home-logo-white-bg
Feb 20, 2026
Merged

fix: 대학 정보 표기의 CURRENT_TERM env 의존 제거#456
manNomi merged 2 commits intomainfrom
fix/university-home-logo-white-bg

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 20, 2026

Summary

  • 대학 카드/상세 페이지에서 NEXT_PUBLIC_CURRENT_TERM 비교 분기를 제거하고 서버 응답의 koreanName 값을 그대로 사용하도록 정리했습니다.
  • 더 이상 사용하지 않는 NEXT_PUBLIC_CURRENT_TERM 항목을 apps/web/.env에서 제거했습니다.
  • 환경변수 가이드 문서에서 CURRENT_TERM 언급을 삭제해 현재 설정과 문서를 일치시켰습니다.

Changed Files

  • apps/web/src/components/ui/UniverSityCard/index.tsx
  • apps/web/src/app/university/[homeUniversity]/[id]/page.tsx
  • apps/web/.env
  • .env.guide.md

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-connect-web-admin Ready Ready Preview, Comment Feb 20, 2026 3:10pm
solid-connection-web Building Building Preview, Comment Feb 20, 2026 3:10pm

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

coderabbitai bot commented Feb 20, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

이번 변경 사항은 학기 정보(CURRENT_TERM) 관련 로직을 체계적으로 제거하는 일관된 리팩토링입니다. 다음과 같이 정리됩니다.

  1. 환경 설정 정리

    • .env.guide.md에서 CURRENT_TERM을 공통 설정 목록에서 제거
    • apps/web/.env에서 NEXT_PUBLIC_CURRENT_TERM 환경 변수 삭제
  2. UI 컴포넌트 학기 표시 제거

    • HomeUniversityCard.tsx의 아바타 컨테이너를 중립적 흰 테두리 스타일로 변경
    • university/[homeUniversity]/[id]/page.tsx에서 페이지 메타데이터 및 본문의 조건부 학기 접미사 제거
    • UniverSityCard/index.tsx에서 표시 로직의 조건부 학기 접미사 제거

전체적으로 환경 변수 의존성을 제거하고 대학 이름 표시를 단순화하는 변경입니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
✨ 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-home-logo-white-bg

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.

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