Skip to content

✨ 중앙대학교 홈 대학 추가 - #622

Merged
manNomi merged 1 commit into
mainfrom
feat/add-chungang-home-university
Aug 3, 2026
Merged

✨ 중앙대학교 홈 대학 추가#622
manNomi merged 1 commit into
mainfrom
feat/add-chungang-home-university

Conversation

@manNomi

Copy link
Copy Markdown
Contributor

요약

서버에 중앙대학교가 홈 대학으로 추가된 것을 apps/university-web에 반영합니다. /university/chungang 목록·검색 페이지와 /university/chungang/:id 상세 페이지가 SSG로 생성됩니다.

서버 데이터 확인

공개 API(/univ-apply-infos/search/text)로 실제 값을 확인하고 반영했습니다.

homeUniversityId홈 대학등록 대학 수
2경희대학교170
3중앙대학교186

변경 내용

  • types/university.tsHomeUniversityNameCHUNGANG = "중앙대학교" 추가, HomeUniversitySlug"chungang" 추가
  • constants/university.tsHOME_UNIVERSITY_SLUG_MAP / HOME_UNIVERSITY_TO_SLUG_MAP / HOME_UNIVERSITY_LIST / HOME_UNIVERSITY_SLUGS에 중앙대 항목 추가 (homeUniversityId: 3, slug chungang, shortName 중앙대)
  • public/images/univs/chungang.png — 로고 추가 (600×600, RGBA, 투명 배경)

color는 로고에서 추출한 CAU 블루 #006DB4를 사용했습니다.

로고 관련 특이사항

전달받은 원본 로고 파일에 빨간색 사각 테두리(주석용 표식으로 보임)가 실제로 포함되어 있었습니다. PNG를 직접 디코딩해 확인한 결과 (228,31,31) 계열 2,530픽셀이 x422~515 / y173~267 영역에 사각 테두리 형태로 존재했고, 테두리가 u 글자 위를 가로지르고 있었습니다.

배경이 투명하고 테두리가 얇아, 주변 색상 기반 인페인트로 제거한 뒤 커밋했습니다. 처리 후 빨강 계열 픽셀 0개이며 남은 색상은 CAU 블루 (0,109,180) 단색입니다. u 획도 끊김 없이 이어지는 것을 확대 비교로 확인했습니다.

공식 CI 원본 파일이 따로 있다면 교체를 권장합니다.

범위 밖 (후속 필요)

apps/web에는 이번 PR에서 추가하지 않았습니다. apps/webHomeUniversityInfo에는 maxChoiceCount(지원 가능 대학 수) 필드가 필수인데, 인하대=3 / 경희대=5로 값이 서로 달라 중앙대 값을 임의로 정할 수 없었습니다. 관리자 API는 인증이 필요해 조회하지 못했습니다.

  • 현재 동작에 회귀는 없습니다. getHomeUniversityById(3)undefined를 반환해 DEFAULT_MAX_CHOICE_COUNT(3)로 폴백하며, 이는 기존 동작과 동일합니다.
  • 중앙대의 실제 maxChoiceCount를 알려주시면 apps/web(홈 대학 목록·sitemap·지원 플로우) 반영을 후속 PR로 올리겠습니다.

검증

  • pnpm --filter @solid-connect/university-web run lint:check — 319 files 통과
  • pnpm --filter @solid-connect/university-web run typecheck:ci — 통과
  • SSG 빌드(run build)는 이번에 실행하지 않았습니다. 머지 전 확인을 권장합니다 — generateStaticParams가 중앙대 186개 대학의 상세 페이지를 새로 생성하며, 이 앱은 데이터 fetch 실패 시 빈 카탈로그 대신 빌드가 실패하도록 설계되어 있습니다.

🤖 Generated with Claude Code

@manNomi
manNomi requested a review from wibaek as a code ownerAugust 3, 2026 14:16
@vercel

vercelBot commented Aug 3, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
solid-connect-university-webBuildingBuildingPreviewAug 3, 2026 2:16pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
solid-connect-web-adminSkippedSkippedAug 3, 2026 2:16pm
solid-connection-webSkippedSkippedAug 3, 2026 2:16pm

@github-actionsgithub-actionsBot added the university Changes in apps/university-web label Aug 3, 2026
@manNomi
manNomi merged commit 67d3dc0 into mainAug 3, 2026
10 of 12 checks passed
@coderabbitai

coderabbitaiBot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 265294a0-1724-45f1-ae35-08d4da5a7bb1

📥 Commits

Reviewing files that changed from the base of the PR and between 46f86c0 and e3155d9.

⛔ Files ignored due to path filters (1)
  • apps/university-web/public/images/univs/chungang.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • apps/university-web/src/constants/university.ts
  • apps/university-web/src/types/university.ts

Walkthrough

  1. 중앙대학교 타입 추가
    HomeUniversityNameHomeUniversitySlug에 중앙대학교 정보를 추가했습니다.

  2. 중앙대학교 설정 등록
    중앙대학교의 양방향 슬러그 매핑, 목록 정보, 유효 슬러그를 추가했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers:wibaek, enunsnv, yoonc01

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/add-chungang-home-university

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.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

universityChanges in apps/university-web

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@manNomi