Skip to content

멘토 체크 API 호출 최적화 및 RootModal 클라이언트 통합#424

Merged
manNomi merged 5 commits intomainfrom
fix/mentor-check-modal-flow
Feb 14, 2026
Merged

멘토 체크 API 호출 최적화 및 RootModal 클라이언트 통합#424
manNomi merged 5 commits intomainfrom
fix/mentor-check-modal-flow

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Feb 14, 2026

Summary

  • auth 전역 스토어에 userRole 상태를 추가해 멘토 여부 판단을 전역 상태 기반으로 통일했습니다.
  • 멘토 신규 신청 카운트 API 호출을 멘토 권한 사용자에만 실행되도록 조건을 수정하고, 멘토 페이지 이동 동작을 보강했습니다.
  • MentorApplyCountModal 렌더링을 RootModalClientModal로 통합하고 ServerModal을 제거했습니다.

Verification

  • pnpm run typecheck 통과
  • pnpm run build 통과

@vercel
Copy link

vercel bot commented Feb 14, 2026

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

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Error Error Feb 14, 2026 1:09pm
solid-connection-web Error Error Feb 14, 2026 1:09pm

@github-actions github-actions bot added the web label Feb 14, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Walkthrough

  1. RootModal 정리. ServerModal 임포트와 렌더링을 제거하고 ClientModal만 렌더링합니다.
  2. ClientModal 확장. MentorApplyCountModal을 추가해 IconConfirmModal과 함께 배열로 렌더링합니다.
  3. ServerModal 삭제. 서버 측 로그인 기반 조건부 렌더링 모듈 전체를 제거했습니다.
  4. 인증 상태 확장. useAuthStore에 userRole: UserRole | null을 추가하고 토큰 파싱으로 userRole을 설정·갱신합니다.
  5. 인증 체크 변경. 여러 컴포넌트에서 토큰 파싱 대신 useAuthStore의 userRole과 isAuthenticated를 사용하도록 전환했습니다.
  6. 네비게이션 변경. MentorApplyCountContent에서 Link 대신 라우터 기반 네비게이션과 버튼 클릭으로 이동하도록 변경했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 분

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 주요 변경 사항을 명확하게 요약합니다: 멘토 체크 API 호출 최적화 및 RootModal 클라이언트 통합.
Description check ✅ Passed PR 설명이 핵심 변경 사항, 검증 정보를 포함하며 요구되는 필수 섹션들을 충분히 다루고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/mentor-check-modal-flow

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
apps/web/src/components/mentor/MentorApplyCountContent/index.tsx (1)

19-19: 얼리 리턴 가드 조건이 논리적으로 충분합니다.

!isAuthenticated 체크가 명시적으로 없지만, 인증되지 않은 사용자는 userRolenull이므로 !isMentor에서 걸립니다. 현재 로직으로 충분하나, 가독성을 위해 !isAuthenticated를 명시적으로 추가하는 것도 고려해볼 수 있습니다 (선택 사항).


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 and usage tips.

@manNomi manNomi merged commit 927005c into main Feb 14, 2026
8 of 10 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