Skip to content

feat: 게스트 로그인 시 약관 동의 화면 스킵 - #241

Merged
whghdus merged 1 commit into
devfrom
feat/238-guest-login-skip-agreement
Sep 19, 2026
Merged

whghdus merged 1 commit into
devfrom
feat/238-guest-login-skip-agreement

Conversation

@whghdus

@whghdus whghdus commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • 게스트(로그인 없이 테스트하기) 로그인 성공 시 약관 동의 화면(/agreement) 대신 온보딩(/onboarding)으로 바로 이동
  • 이후 홈에서 약관 재동의 바텀시트가 다시 뜨지 않도록, 필수 약관에 대해 백그라운드로 자동 동의 처리
  • 카카오/네이버 로그인 플로우는 기존 약관 동의 화면 그대로 유지

Test plan

  • pnpm run typecheck
  • pnpm run lint
  • pnpm test
  • 배포 환경에서 게스트 로그인 → 온보딩 직행 및 홈 진입 후 약관 재동의 바텀시트 미노출 확인 (로컬 dev 프록시는 백엔드 Origin 검증으로 차단되어 확인 불가)

Closes #238

🤖 Generated with Claude Code

Summary by CodeRabbit

  • 새로운 기능
    • 게스트 로그인으로 처음 가입하는 경우 필수 약관에 자동 동의하고 온보딩으로 바로 이동합니다.
    • 약관 조회 또는 동의 처리에 실패하더라도 온보딩을 계속 진행할 수 있습니다.
    • 일반 로그인 흐름에서는 기존과 동일하게 약관 동의 화면이 표시됩니다.

- 게스트(로그인 없이 테스트하기) 로그인 성공 시 약관 동의 화면(/agreement) 대신 온보딩으로 바로 이동
- 이후 홈에서 약관 재동의 바텀시트가 다시 뜨지 않도록, 필수 약관에 대해 백그라운드로 자동 동의 처리
- 카카오/네이버 로그인 플로우는 기존 약관 동의 화면 그대로 유지

Closes #238

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

vercel Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
brifo-front Ready Ready Preview Sep 19, 2026 11:54am UTC

@coderabbitai

coderabbitai Bot commented Sep 19, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: Team-BRIFO/BRIFO-front/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: 99d07bcf-1dde-4417-8ed9-7e5dbca1135a

📥 Commits

Reviewing files that changed from the base of the PR and between 7023698 and 1b95618.

📒 Files selected for processing (2)
  • src/pages/SplashPage/SplashPage.tsx
  • src/services/auth/oauthLoginResult.ts

Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

게스트 로그인은 skipAgreement: true로 약관 화면을 건너뜁니다. 필수 약관 자동 동의를 시도한 뒤 온보딩으로 이동합니다. 자동 동의에 실패해도 온보딩으로 이동하며, 일반 OAuth 로그인 흐름은 기존 경로를 유지합니다.

Changes

게스트 로그인 온보딩 흐름

Layer / File(s) Summary
약관 자동 동의 및 이동 처리
src/services/auth/oauthLoginResult.ts
applyOAuthLoginResult에 skipAgreement 옵션을 추가했습니다. 옵션이 활성화되면 필수 약관을 조회하고 자동 동의를 시도합니다. 처리에 실패해도 온보딩으로 이동합니다.
게스트 로그인 옵션 연결
src/pages/SplashPage/SplashPage.tsx
게스트 로그인 결과 처리 시 skipAgreement: true를 전달합니다. 일반 로그인은 기본값으로 기존 약관 동의 화면을 사용합니다.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 1b956

Guest users continue to onboarding if background policy agreement fails, and any remaining required agreement is presented after entering the app. No actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 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 직접 연결된 이슈 #238의 코딩 요구사항을 충족합니다. SplashPage는 게스트 로그인 성공 시 applyOAuthLoginResult(..., { skipAgreement: true })를 호출합니다. applyOAuthLoginResult는 해당 옵션에서 GET /api/policies에 해당하는 getPolicies와 `POST…
Out of Scope Changes check ✅ Passed 변경은 SplashPage의 게스트 로그인 분기와 applyOAuthLoginResult의 게스트 약관 자동 동의 분기에 한정됩니다. 두 변경은 모두 이슈 #238의 게스트 온보딩 및 기존 소셜 로그인 유지 요구사항과 직접 연결됩니다. 관련 없는 기능 변경은 확인되지 않습니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@whghdus
whghdus merged commit 9a74310 into dev Sep 19, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 1b956188 Deployed Sep 19, 2026 by vercel[bot]
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.

[Feat] 게스트(로그인 없이 테스트하기) 온보딩 약관 동의 화면 스킵

1 participant