Skip to content

fix: 지원자 현황 대학 수 단위 표시#604

Merged
manNomi merged 1 commit into
mainfrom
fix/application-score-count-unit
Jul 9, 2026
Merged

fix: 지원자 현황 대학 수 단위 표시#604
manNomi merged 1 commit into
mainfrom
fix/application-score-count-unit

Conversation

@manNomi

@manNomi manNomi commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

  • 없음

작업 내용

  • 지원자 현황 확인 페이지의 탭 카운트가 학교 수임을 명확히 알 수 있도록 단위를 추가했습니다.
  • 모든 대학, 지원자 있는 대학 모두 인원수가 아니라 대학 개수 기준임을 UI 문구로 드러냈습니다.

특이 사항

  • 기능 동작 변경 없이 표시 문구만 수정했습니다.

리뷰 요구사항 (선택)

  • 탭 카운트 단위가 사용자에게 자연스럽게 읽히는지 확인 부탁드립니다.

검증

  • pnpm --filter @solid-connect/web run ci:check
  • push hook: web ci:check 및 build 통과

@manNomi manNomi requested a review from wibaek as a code owner July 9, 2026 10:24
@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 Building Building Preview, Comment Jul 9, 2026 10:25am
solid-connection-web Ready Ready Preview, Comment Jul 9, 2026 10:25am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 9, 2026 10:25am

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: fe6426a3-8859-43c0-8b62-c3b1d43bcac5

📥 Commits

Reviewing files that changed from the base of the PR and between 1123bff and dee6303.

📒 Files selected for processing (1)
  • apps/web/src/app/university/application/ScorePageContent.tsx

Walkthrough

  1. 대학 탭 카운트 표기 방식 변경
    • "모든 대학" 탭과 "지원자 있는 대학" 탭에 표시되던 숫자 옆에 "개" 단위가 추가되었습니다.
    • 기존: ({count}) → 변경: ({count}개)
    • 다른 로직이나 export되는 항목의 변경은 없습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: wibaek

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 대학 수에 단위를 표시하는 UI 변경을 정확히 요약해 변경 내용과 잘 맞습니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 특이 사항, 리뷰 요구사항, 검증까지 템플릿의 주요 항목을 대부분 채웠습니다.
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.
✨ 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-count-unit

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.

@manNomi manNomi merged commit e1c3eb0 into main Jul 9, 2026
5 of 7 checks passed
@github-actions github-actions Bot added the web label Jul 9, 2026
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