Skip to content

fix: 지원자 현황 지원 대학 표시 수정#605

Merged
manNomi merged 2 commits into
mainfrom
fix/application-score-applied-university-ui
Jul 11, 2026
Merged

fix: 지원자 현황 지원 대학 표시 수정#605
manNomi merged 2 commits into
mainfrom
fix/application-score-applied-university-ui

Conversation

@manNomi

@manNomi manNomi commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

  • 없음

작업 내용

  • 지원자 현황 확인 페이지 상단 문구를 지원한 대학 ({n}개) 형식으로 수정했습니다.
  • 지원한 대학 카드 우측 꺾새를 문자 표시에서 기존 SVG 아이콘으로 교체해 리스트 카드와 시각적으로 통일했습니다.
  • 서버에서 내려주는 대학 logoImageUrl을 지원한 대학 카드와 지원자 현황 목록 카드에 표시하도록 반영했습니다.
  • 동일 대학 데이터 병합 시 logoImageUrl, backgroundImageUrl이 누락되지 않도록 보강했습니다.

특이 사항

  • 기능 동작 변경 없이 UI 표시 문구와 이미지 렌더링만 수정했습니다.
  • /applications API는 인증이 필요해 비인증 curl에서는 401이 반환됩니다. 현재 프론트 타입에 반영된 logoImageUrl/backgroundImageUrl 기준으로 렌더링을 연결했습니다.
  • 기존 PR #604가 이미 머지되어, 이번 수정은 별도 브랜치/PR로 분리했습니다.

리뷰 요구사항 (선택)

  • 지원한 대학 카운트 단위와 카드 우측 아이콘이 디자인 의도와 맞는지 확인 부탁드립니다.
  • 지원자 현황 목록에서 대학 로고가 의도한 크기로 보이는지 확인 부탁드립니다.

검증

  • pnpm --filter @solid-connect/web exec biome check src/app/university/application/ScoreSheet.tsx src/app/university/application/ScorePageContent.tsx 'src/app/university/application/[universityName]/ApplicationUniversityDetailContent.tsx' src/types/application.ts
  • pnpm --filter @solid-connect/web run typecheck:ci
  • commit hook: web ci:check 통과
  • push hook: web ci:checkbuild 통과

@manNomi manNomi requested a review from wibaek as a code owner July 9, 2026 11:14
@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
solid-connect-university-web Ready Ready Preview, Comment Jul 9, 2026 12:37pm
solid-connection-web Ready Ready Preview, Comment Jul 9, 2026 12:37pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 9, 2026 12:37pm

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

  1. 타입 손질
    • ScoreSheet의 logoImageUrl, backgroundImageUrl 타입이 null도 받아줄 수 있게 넓어졌어요.
  2. 새 얼굴, ScoreSheetLogo 등장
    • FallbackImage 기반의 ScoreSheetLogo 컴포넌트가 새로 생겼고, MobileScoreSheet의 로고 자리도 이 친구로 교체됐어요.
  3. 지원한 대학 목록 단장
    • 목록 항목이 밋밋한 원형 더미 대신 ScoreSheetLogo와 IconExpandMoreFilled로 산뜻하게 바뀌었고, 헤딩 문구도 살짝 다듬어졌어요.
  4. 중복 스코어시트, 이제 꼼꼼하게 병합
    • 중복 제거 시 applicants만 합치던 걸, englishName·logoImageUrl·backgroundImageUrl까지 우선순위대로 채워 넣도록 보강했어요.
  5. 대학 상세 페이지 이미지도 야무지게
    • matches 배열을 훑어 logoImageUrl과 backgroundImageUrl을 최대한 채운 mergedUniversityImages를 만들어 최종 결과에 반영해요.

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

Suggested reviewers: wibaek, enunsnv

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed 제목이 지원자 현황의 지원 대학 표시와 아이콘/표시 수정이라는 주요 변경점을 간결하게 요약합니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 특이 사항, 리뷰 요구사항이 모두 포함되어 있어 템플릿을 충분히 충족합니다.
✨ 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 fix/application-score-applied-university-ui

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.

@vercel vercel Bot temporarily deployed to Preview – solid-connect-web-admin July 9, 2026 12:35 Inactive
@manNomi manNomi merged commit fe3aaa8 into main Jul 11, 2026
12 of 13 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