Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat] 관리자: 회원 탈퇴 기능 추가 #61

Merged
merged 20 commits into from
Dec 26, 2024

Conversation

limseohyeon
Copy link
Collaborator

@limseohyeon limseohyeon commented Dec 26, 2024

#️⃣ 연관된 이슈

#42

✅ 체크리스트

  • 🔀 PR 제목의 형식 맞추기 ex) [feat] ㅇㅇ 기능 추가
  • 💡 이슈 등록
  • 🏷️ 라벨 등록
  • 🧹 불필요한 코드 제거
  • 🧪 로컬 테스트 완료
  • 🏗️ 빌드 성공
  • 💯 테스트 통과

📝 작업 내용

사용자 Hard delete 구현했습니다. 내용은 아래와 같습니다.

  1. 상태가 WITHDRAWAL(탈퇴)
  2. 사용자 정보 수정 일이 현재 날짜 기준 6개월 전인 사용자
  3. 매일 23:59분에 진행

📸 스크린샷 (UI 변경 시 필수)

👀 리뷰어 가이드라인

여기에 작성

Copy link

github-actions bot commented Dec 26, 2024

Test Results

31 tests   31 ✅  5s ⏱️
12 suites   0 💤
12 files     0 ❌

Results for commit e0a0929.

♻️ This comment has been updated with latest results.

@DongminL DongminL assigned DongminL and limseohyeon and unassigned DongminL Dec 26, 2024
@DongminL
Copy link
Collaborator

현재 올려주신 서브 모듈이 최신 버전이 아니라서 문제가 생긴 거 같아요.
서브 모듈도 업데이트하셔서 다시 올리시면 될 것 같습니다.
그리고 지금처럼 CI 작업이 실패하면, Actions에서 어디서 에러 났는지 확인할 수 있습니다.

@limseohyeon limseohyeon linked an issue Dec 26, 2024 that may be closed by this pull request
1 task

@Configuration
@RequiredArgsConstructor
@EnableBatchProcessing
Copy link
Collaborator

@DongminL DongminL Dec 26, 2024

Choose a reason for hiding this comment

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

@EnableBatchProcessing 어노테이션을 빼야 yml 파일의 설정대로 동작합니다

Copy link
Collaborator

@DongminL DongminL Dec 26, 2024

Choose a reason for hiding this comment

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

제가 지우고 머지할게요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 알겠습니다 감사해요...! 너무 늦게 봤네요 ,,,

@DongminL DongminL self-assigned this Dec 26, 2024
@DongminL DongminL merged commit 04c5c68 into main Dec 26, 2024
3 checks passed
@DongminL DongminL deleted the feature/#42-관리자-회원관리-삭제 branch December 26, 2024 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

관리자 : 회원관리 삭제
2 participants