Skip to content

fix: 스테이지 환경에서는 탈퇴 유예 무효화#335

Merged
dh2906 merged 1 commit intodevelopfrom
fix/user-withdraw-restore-stage
Feb 28, 2026
Merged

fix: 스테이지 환경에서는 탈퇴 유예 무효화#335
dh2906 merged 1 commit intodevelopfrom
fix/user-withdraw-restore-stage

Conversation

@dh2906
Copy link
Contributor

@dh2906 dh2906 commented Feb 28, 2026

🔍 개요

  • close #이슈번호

🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@dh2906 dh2906 self-assigned this Feb 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9168d00 and 7db25f7.

📒 Files selected for processing (1)
  • src/main/java/gg/agit/konect/global/auth/oauth/OAuthLoginHelper.java

📝 Walkthrough

요약

OAuthLoginHelper에 stage 프로필 인식 제어 기능을 도입합니다. Environment 프로필 확인 및 isStageProfile() 헬퍼 메서드를 추가하여 stage 프로필이 활성화되었을 때 복원 경로를 단락시킵니다.

변경 사항

코호트 / 파일 요약
Stage 프로필 기반 조건부 제어
src/main/java/gg/agit/konect/global/auth/oauth/OAuthLoginHelper.java
STAGE_PROFILE 상수 및 Environment 의존성을 추가하고, 스테이지 환경에서 계정 복원 로직(restoreWithdrawnByProviderId, restoreWithdrawnByEmail)을 조건부로 비활성화하는 isStageProfile() 헬퍼 메서드 구현

예상 코드 리뷰 소요 시간

🎯 2 (Simple) | ⏱️ ~8분

🐰 Stage의 문을 지키는 환경 프로필,
복원의 길을 슬기롭게 가르쳐주네.
무대 위 계정들은 소중히 보호되고,
프로덕션은 안전하게 흐른다.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/user-withdraw-restore-stage

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

@dh2906 dh2906 merged commit 31cd6c1 into develop Feb 28, 2026
1 of 2 checks passed
@dh2906 dh2906 deleted the fix/user-withdraw-restore-stage branch February 28, 2026 16:18
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