Skip to content

[Bugfix/#379] 회원가입 완료 후 로그인 페이지로 이동되지 않는 문제 해결 - #380

Merged
jjjsun merged 1 commit into
developfrom
bugfix/#379
Aug 3, 2026
Merged

[Bugfix/#379] 회원가입 완료 후 로그인 페이지로 이동되지 않는 문제 해결#380
jjjsun merged 1 commit into
developfrom
bugfix/#379

Conversation

@jjjsun

@jjjsun jjjsun commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

🚨 관련 이슈

Closed #379

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

  • 회원가입 프로필 단계 성공시에 resetAuth()를 먼저 호출하게 되면서, store의 email이 비워지고, 그다음 navigate(buildPathWithReturnUrl("/login", returnUrl) 단계에서 같은 컴포넌트의 guard useEffect!email을 먼저 감지해서 /signup으로 다시 이동하는 에러를 수정했습니다.
  • 성공 콜백에서는 login으로만 이동하고(returnUrl이 있다면 그 Url로 이동), emailsocialId 초기화는 Signup페이지에서 resetAuth 실행됩니다.

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

  • resetAuth()는 폼 전체 지우는 것이 아니라 Zustand의 email, socialId`만 초기화합니다. 나머지 입력값은 페이지 언마운트시 함께 사라지는 구조입니다.
  • 기존 !email || !password은 그대로 동작 유지됩니다.

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

  • 버그 수정
    • 회원가입 프로필 설정 완료 후 인증 상태가 불필요하게 초기화되던 문제를 수정했습니다.
    • 회원가입 성공 알림 표시 후 로그인 페이지로 이동하는 흐름은 유지됩니다.

@jjjsun
jjjsun requested review from Seojegyeong and YermIm August 3, 2026 03:15
@jjjsun jjjsun self-assigned this Aug 3, 2026
@jjjsun jjjsun added the 🐞 BugFix Something isn't working label Aug 3, 2026
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

회원가입 프로필 설정 완료 시 resetAuth 호출을 제거했습니다. 성공 토스트 표시 후 로그인 페이지로 이동하는 흐름은 유지됩니다.

Changes

회원가입 완료 처리

Layer / File(s) Summary
성공 처리 및 로그인 이동
src/components/auth/flows/signup/ProfileSetupStep.tsx
useAuthStore에서 resetAuth 조회를 제거했습니다. 회원가입 성공 후 resetAuth() 호출을 제거하고 로그인 페이지 이동은 유지했습니다.

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

Suggested reviewers: yermim

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed [379]의 로그인 이동, returnUrl 처리, 기존 가드 유지 요구사항을 변경 내용과 설명에서 모두 충족합니다.
Out of Scope Changes check ✅ Passed 변경은 ProfileSetupStep의 불필요한 resetAuth 호출 제거로 제한되며, 연결된 버그 수정 범위를 벗어나지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed 제목이 회원가입 완료 후 로그인 페이지로 이동하지 않는 버그 수정이라는 주요 변경사항을 명확하게 설명합니다.
Description check ✅ Passed 관련 이슈, 변경 유형, 작업 내용, 미완성 작업, 참고 사항을 포함해 템플릿의 필수 내용을 충실히 작성했습니다.
✨ 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 bugfix/#379

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.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

📚 Storybook 배포 완료

항목 링크
📖 Storybook https://69a147b60a56365d9e2185ef-yixexslrwx.chromatic.com/
🔍 Chromatic https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=530

@Seojegyeong Seojegyeong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P4: 확인했습니다!

@jjjsun
jjjsun merged commit 1a4242a into develop Aug 3, 2026
3 checks passed
@jjjsun
jjjsun deleted the bugfix/#379 branch August 3, 2026 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 BugFix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞 [BugFix] 회원가입 완료 후 로그인 페이지로 이동되지 않는 에러

2 participants