Skip to content

[Refactor] 유저 상태 조회에 약관 동의 여부 필드 추가#224

Merged
jeong1112 merged 1 commit intodevelopfrom
refactor/#223-agree-term
Mar 31, 2026
Merged

[Refactor] 유저 상태 조회에 약관 동의 여부 필드 추가#224
jeong1112 merged 1 commit intodevelopfrom
refactor/#223-agree-term

Conversation

@jeong1112
Copy link
Copy Markdown
Collaborator

@jeong1112 jeong1112 commented Mar 31, 2026

Related issue 🛠

Work Description 📝

  • 작업 내용

ScreenShots 📷

To Reviewers 📢

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 회원 상태 조회 시 약관 동의 여부 정보가 포함되도록 업데이트됨

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c59a67fb-fb5c-403c-88b9-ffb144458c65

📥 Commits

Reviewing files that changed from the base of the PR and between a9fd2b1 and 07e1945.

📒 Files selected for processing (3)
  • src/main/java/org/sopt/kareer/domain/member/dto/response/MemberStatusResponse.java
  • src/main/java/org/sopt/kareer/domain/member/repository/MemberTermRepository.java
  • src/main/java/org/sopt/kareer/domain/member/service/MemberService.java

📝 Walkthrough

Walkthrough

사용자 정보 조회 응답에 약관 동의 여부를 추가합니다. 응답 DTO에 필드를 추가하고, 리포지토리에 동의 여부 확인 메서드를 추가한 후, 서비스에서 해당 정보를 조회하여 응답에 포함시킵니다.

Changes

Cohort / File(s) Summary
응답 DTO 업데이트
src/main/java/org/sopt/kareer/domain/member/dto/response/MemberStatusResponse.java
agreedTerm 필드 추가 및 from() 메서드 시그니처 업데이트. 약관 동의 여부를 응답에 포함시키도록 수정.
저장소 쿼리 메서드 추가
src/main/java/org/sopt/kareer/domain/member/repository/MemberTermRepository.java
existsByMemberIdAndAgreedTrue(Long memberId) 메서드 추가. 회원의 약관 동의 여부를 데이터베이스에서 조회.
서비스 로직 수정
src/main/java/org/sopt/kareer/domain/member/service/MemberService.java
getMemberStatus() 메서드에서 memberTermRepository를 통해 약관 동의 여부를 조회하고 응답 생성시 전달.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 약관 동의, 이제 물어보네
조회할 때마다 확인 착착
저장소는 진실을 알고 있고
응답에 담아 건넨다네

✨ 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 refactor/#223-agree-term

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 and usage tips.

@jeong1112 jeong1112 merged commit 992296a into develop Mar 31, 2026
1 of 2 checks passed
@jeong1112 jeong1112 deleted the refactor/#223-agree-term branch March 31, 2026 11:25
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.

[Refactor] 사용자 정보 조회에 약관 동의 여부 추가

1 participant