Skip to content

fix: 파견 가능학기 결측값 표시 수정#602

Open
manNomi wants to merge 1 commit into
mainfrom
fix/dispatch-semester-fallback
Open

fix: 파견 가능학기 결측값 표시 수정#602
manNomi wants to merge 1 commit into
mainfrom
fix/dispatch-semester-fallback

Conversation

@manNomi

@manNomi manNomi commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

  • 없음

작업 내용

  • 파견 학교 상세의 파견 가능학기 값이 없을 때 데이터 없음 또는 NO_DATA가 그대로 노출되지 않도록 표시값을 정규화했습니다.
  • 결측값은 상세 화면에서 -로 표시되도록 변경했습니다.

검증

  • pnpm --filter @solid-connect/university-web run lint:check
  • pnpm --filter @solid-connect/university-web run typecheck:ci
  • NEXT_PUBLIC_API_SERVER_URL=https://api.solid-connection.com NEXT_PUBLIC_WEB_URL=https://www.solid-connection.com pnpm --filter @solid-connect/university-web run build
  • 빌드 산출물에서 파견 가능학기 값이 -로 표시되는 것 확인
  • commit/push hook의 CI parity checks 통과

특이 사항

  • API/관리자 데이터 값은 변경하지 않고, 사용자 상세 화면의 표시값만 정리했습니다.

리뷰 요구사항 (선택)

  • 결측값 표시를 -로 통일하는 방향이 상세 화면 UX에 적절한지 확인 부탁드립니다.

@manNomi manNomi requested a review from wibaek as a code owner July 8, 2026 08:20
@vercel

vercel Bot commented Jul 8, 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 8, 2026 8:21am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 8, 2026 8:21am
solid-connection-web Skipped Skipped Jul 8, 2026 8:21am

@coderabbitai

coderabbitai Bot commented Jul 8, 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: 4835d193-390f-4e12-8466-a9845b18fad9

📥 Commits

Reviewing files that changed from the base of the PR and between 213318d and 4badc87.

📒 Files selected for processing (1)
  • apps/university-web/src/app/university/[homeUniversity]/[id]/_ui/UniversityDetail/_ui/InfoSection.tsx

Walkthrough

  1. 파견 가능학기 값 포맷팅 로직 도입
    • 값이 공백이거나 NO_DATA, 데이터 없음인 경우 "-"로 치환하는 formatSemesterAvailableForDispatch 헬퍼 함수를 새로 추가했어요.
  2. 컴포넌트 내 라벨 계산 적용
    • semesterAvailableForDispatchLabel 변수를 만들어 헬퍼 함수의 결과를 저장하도록 했어요.
  3. 화면 렌더링 값 교체
    • 기존에 원본 값을 그대로 보여주던 부분을 새로 계산한 라벨 값으로 바꿔서 화면에 표시하도록 했어요.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 PR의 주요 변경 사항을 명확하고 간결하게 요약하고 있습니다. 파견 가능학기 결측값 표시 수정이라는 핵심 내용을 정확히 전달합니다.
Description check ✅ Passed PR 설명이 템플릿의 필수 섹션(관련 이슈, 작업 내용, 특이 사항)을 충실히 포함하고 있으며, 추가로 검증 내용과 리뷰 요구사항도 구체적으로 제시하고 있습니다.
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/dispatch-semester-fallback

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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

university Changes in apps/university-web

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant