Skip to content

Comments

본인인증 풀리는 문제 수정#382

Merged
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/verify
Feb 7, 2026
Merged

본인인증 풀리는 문제 수정#382
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
fix/verify

Conversation

@HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Feb 7, 2026

🌱 연관된 이슈

☘️ 작업 내용

  • localStorage에서 관리되고 있던 본인인증 상태를 서버 상태를 통해 관리하도록 수정

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

  • 리팩터
    • 휴대폰 인증 상태 관리를 개선했습니다. 서버와의 동기화를 강화하여 인증 상태의 일관성을 보장하고 UI 깜빡임을 방지합니다. 전체 앱에서 휴대폰 인증이 더욱 안정적으로 작동합니다.

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Feb 7, 2026
@vercel
Copy link

vercel bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment Feb 7, 2026 2:56am

@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

📝 Walkthrough

Walkthrough

전화 인증 상태를 관리하는 새로운 usePhoneVerificationStatus 훅이 도입되어 서버 데이터와 로컬 Zustand 저장소를 동기화합니다. 기존 usePhoneVerificationStore 훅을 사용하던 여러 컴포넌트가 새 훅으로 마이그레이션되고 있습니다.

Changes

Cohort / File(s) Summary
Phone Verification 훅 도입
src/features/phone-verification/model/use-phone-verification-status.ts
usePhoneVerificationStatus 훅 신규 구현. 서버 인증 상태와 로컬 Zustand 저장소 동기화, 메모이제이션 적용, 선택적 memberId 오버라이드 지원, 기존 훅의 하위호환성 재내보내기 포함
Phone Verification 마이그레이션
src/components/home/study-matching-toggle.tsx, src/features/study/group/ui/apply-group-study-modal.tsx, src/features/study/group/ui/group-study-form-modal.tsx, src/features/study/participation/ui/reservation-list.tsx, src/features/study/participation/ui/start-study-modal.tsx, src/widgets/home/todo-list.tsx
기존 usePhoneVerificationStoreusePhoneVerificationStatus로 치환. 서버 동기화 상태 사용 안내 주석 추가. 일부 컴포넌트에서 memberId 파라미터 추가
타입 정제
src/entities/user/model/use-user-profile-query.ts
onMutate에서 타입 캐스팅 제거: ...(prev as any)에서 ...prev로 변경. TypeScript 타입 추론 활용

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~35 minutes

Poem

🐰 새로운 훅이 피어나고,
서버와 손잡고 춤을 춘다네.
옛날 가게는 안녕히,
진리의 상태가 빛난다.
토끼는 박수를 친다! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 14.29% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 설명합니다. localStorage에서 서버 측 상태 관리로의 인증 상태 전환이라는 주요 변경 목표를 잘 반영하고 있습니다.

✏️ 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/verify

Comment @coderabbitai help to get the list of available commands and usage tips.

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit 2a84cce into develop Feb 7, 2026
10 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the fix/verify branch February 7, 2026 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant