Skip to content

fix: 프로필 수정 출신학교 표시값 수정#593

Merged
manNomi merged 1 commit into
mainfrom
fix/profile-home-university
Jul 1, 2026
Merged

fix: 프로필 수정 출신학교 표시값 수정#593
manNomi merged 1 commit into
mainfrom
fix/profile-home-university

Conversation

@manNomi

@manNomi manNomi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

  • 없음

작업 내용

  • 마이페이지 프로필 수정 화면의 출신학교 표시값에서 인하대학교 하드코딩 fallback을 제거했습니다.
  • /my 프로필 응답의 homeUniversityName을 출신학교로 표시하도록 변경했습니다.
  • 학교 인증 전처럼 homeUniversityNamenull이거나 누락된 경우 등록된 출신학교가 없습니다로 표시되도록 처리했습니다.
  • 멘토/어드민의 수학 학교는 attendedUniversity를 사용하도록 하드코딩 값을 제거했습니다.

특이 사항

  • 브루노의 /my 응답 예시에는 homeUniversityName이 누락되어 있어, 프론트 타입을 homeUniversityName?: string | null로 맞췄습니다.

리뷰 요구사항 (선택)

  • 프로필 수정 화면에서 학교 인증 완료/미완료 사용자 표시 문구가 의도와 맞는지 확인 부탁드립니다.

검증

  • pnpm --filter @solid-connect/web lint:check
  • pnpm --filter @solid-connect/web typecheck:ci
  • pnpm --filter @solid-connect/web build
  • 표시값 분기 수동 검증: 학교 있음, null, 필드 누락, 멘토 케이스 확인

@manNomi manNomi requested review from enunsnv and wibaek as code owners July 1, 2026 08:05
@vercel

vercel Bot commented Jul 1, 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 1, 2026 8:07am
solid-connection-web Ready Ready Preview, Comment Jul 1, 2026 8:07am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jul 1, 2026 8:07am

@github-actions github-actions Bot added the web label Jul 1, 2026
@coderabbitai

coderabbitai Bot commented Jul 1, 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: a21eb934-b8aa-464d-b7ed-bc7370cd3efa

📥 Commits

Reviewing files that changed from the base of the PR and between d8753d2 and f57ebd0.

📒 Files selected for processing (4)
  • apps/web/src/app/my/modify/_ui/ModifyContent/_hooks/useModifyUserHookform.ts
  • apps/web/src/app/my/modify/_ui/ModifyContent/_lib/profileFields.ts
  • apps/web/src/app/my/modify/_ui/ModifyContent/index.tsx
  • apps/web/src/types/myInfo.ts

Walkthrough

1️⃣ 먼저 myInfo의 타입 계약이 한층 더 야무져졌어요 — Partial<MyInfoResponse>라는 다소 느슨한 옷을 벗고 MyInfoResponse라는 정장으로 갈아입었습니다.

  • BaseUserInfohomeUniversityName도 이제 선택적 속성으로 여유를 얻었습니다.

2️⃣ 그 다음, 새로운 일꾼 resolveModifyProfileReadOnlyFields 함수가 등장했습니다.

  • 역할이 MENTOR 또는 ADMIN일 때만 attendedUniversity를 채워주고, 나머지는 얌전히 빈 문자열을 반환해요.
  • homeUniversityName이 비어있으면 빈 문자열로 다정하게 대신해줍니다.

3️⃣ 마지막으로 ModifyContent 컴포넌트가 이 헬퍼를 불러들여, 하드코딩되어 있던 값들을 실제 계산된 값으로 교체했습니다.

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

Suggested reviewers: wibaek, enunsnv

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 프로필 수정 화면의 출신학교 표시값 변경이라는 핵심 내용을 간결하게 잘 요약합니다.
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/profile-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.

@manNomi manNomi merged commit c794190 into main Jul 1, 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