Skip to content

feat: 회원가입 후 자동 로그인 및 설정 페이지 레이아웃 수정 - #22

Open
sssang12 wants to merge 2 commits into
developfrom
feat/register-autologin-and-setting-layout
Open

sssang12 wants to merge 2 commits into
developfrom
feat/register-autologin-and-setting-layout

Conversation

@sssang12

@sssang12 sssang12 commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

작업 내용

PR #19가 develop에 머지된 이후 같은 브랜치(fix/check-id-endpoint)에 추가로 쌓인 커밋 2개가 반영이 안 되어 있어서, develop 최신 상태에서 새로 브랜치를 따서 이 PR로 올립니다.

  1. 회원가입 성공 시 자동 로그인 후 온보딩으로 바로 이동: 회원가입 API는 토큰을 반환하지 않아서, 가입 성공 직후 같은 아이디/비밀번호로 로그인 API를 이어서 호출해 즉시 로그인 상태로 전환하고 activateOnboarding 값에 따라 /onboarding(또는 /)으로 이동하도록 변경. 자동 로그인이 실패하는 예외 상황에서는 계정은 이미 생성된 상태이므로 /login으로 보내 직접 재시도하게 함
  2. 설정 페이지 상단 여백 제거

확인 사항

  • 실제 신규 계정으로 회원가입 → 바로 /onboarding 이동까지 로컬에서 직접 확인

테스트

  • npm run lint
  • npm run build
  • 로컬에서 회원가입 → 자동 로그인 → 온보딩 이동 확인

Summary by CodeRabbit

  • 새로운 기능

    • 회원가입 성공 후 자동 로그인을 시도합니다.
    • 자동 로그인에 성공하면 온보딩 또는 메인 화면으로 이동합니다.
    • 자동 로그인에 실패하면 로그인 화면으로 이동합니다.
  • 스타일

    • 설정 페이지 상단 여백을 제거해 콘텐츠가 화면 상단부터 표시됩니다.

Sangwon83 and others added 2 commits September 4, 2026 21:57
회원가입 API는 토큰을 반환하지 않아서, 가입 성공 직후 같은 아이디/
비밀번호로 로그인 API를 이어서 호출해 즉시 로그인 상태로 전환하고
activateOnboarding 값에 따라 /onboarding(또는 /)으로 이동하도록 변경.
자동 로그인이 실패하는 예외 상황에서는 계정은 이미 생성된 상태이므로
/login으로 보내 직접 재시도하게 함.

실제 신규 계정으로 회원가입 → 바로 /onboarding 이동까지 확인함.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
vac-client Ready Ready Preview Sep 4, 2026 1:00pm UTC

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 63e9f774-b5bd-4b4e-a49a-5bbd44ea34c3

📥 Commits

Reviewing files that changed from the base of the PR and between de94b9d and 9491a6e.

📒 Files selected for processing (2)
  • src/pages/Register.tsx
  • src/pages/Setting.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

회원가입 성공 후 자동 로그인을 시도하고 결과에 따라 라우팅합니다. 설정 페이지 루트 컨테이너의 상단 패딩을 제거합니다.

Changes

회원가입 후 자동 로그인

Layer / File(s) Summary
회원가입 후 인증 및 라우팅
src/pages/Register.tsx
signup 성공 후 login을 호출합니다. 로그인 성공 시 activateOnboarding 값에 따라 /onboarding 또는 /로 이동합니다. 로그인 실패 시 /login으로 이동합니다.

설정 페이지 레이아웃

Layer / File(s) Summary
설정 페이지 상단 여백 조정
src/pages/Setting.tsx
루트 컨테이너에서 pt-[34px] 클래스를 제거합니다.

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

Merge Risk: ⚪ Minimal · up to 9491a

Successful signup now automatically signs the user in and routes them to onboarding or home, with login fallback on failure; the settings page no longer adds top padding. No merge-blocking risk remains.

Sequence Diagram(s)

sequenceDiagram
  participant Register
  participant signup
  participant login
  participant Router
  Register->>signup: 회원가입 요청
  signup-->>Register: 성공
  Register->>login: id와 password로 자동 로그인
  login-->>Register: 성공 또는 실패
  Register->>Router: 성공 시 /onboarding 또는 / 이동
  Register->>Router: 실패 시 /login 이동
Loading

Suggested reviewers: sangwon83

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 회원가입 후 자동 로그인과 설정 페이지 레이아웃 수정이라는 변경 사항을 정확하고 간결하게 설명합니다.
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.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/register-autologin-and-setting-layout

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.

@sssang12

sssang12 commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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.

2 participants