Skip to content

Feature/onboarding 복구입니다 - #24

Merged
hyeonszz merged 2 commits into
mainfrom
feature/onboarding
Jul 10, 2026
Merged

Feature/onboarding 복구입니다#24
hyeonszz merged 2 commits into
mainfrom
feature/onboarding

Conversation

@JikkeunWoo

@JikkeunWoo JikkeunWoo commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

복구

Summary by CodeRabbit

  • 새로운 기능

    • 사용자 ID로 닉네임, 이메일, 이메일 알림 수신 상태를 조회할 수 있습니다.
    • 이메일 알림 수신 여부를 설정할 수 있습니다.
    • 알림 설정 입력값에 대한 필수 검증이 적용됩니다.
  • 테스트

    • 사용자 정보 조회 및 알림 설정 변경 기능의 정상 동작을 검증하는 테스트를 추가했습니다.

@coderabbitai

coderabbitai Bot commented Jul 10, 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: Pro Plus

Run ID: 4f4c2c33-a77c-4a57-a40b-134d53c19a76

📥 Commits

Reviewing files that changed from the base of the PR and between bd35dde and b32219a.

📒 Files selected for processing (6)
  • src/main/java/com/example/hackathon/domain/user/controller/UserController.java
  • src/main/java/com/example/hackathon/domain/user/dto/request/UserNotificationRequest.java
  • src/main/java/com/example/hackathon/domain/user/dto/response/UserResponse.java
  • src/main/java/com/example/hackathon/domain/user/entity/User.java
  • src/main/java/com/example/hackathon/domain/user/service/UserService.java
  • src/test/java/com/example/hackathon/domain/user/service/UserServiceTest.java

📝 Walkthrough

Walkthrough

사용자 상세 조회와 이메일 알림 설정 변경 기능을 추가했다. 요청·응답 DTO, 서비스 로직, 사용자 엔티티 갱신 메서드, 컨트롤러 엔드포인트 및 서비스 테스트가 포함된다.

Changes

사용자 정보 및 알림 설정 API

Layer / File(s) Summary
사용자 API DTO 계약
src/main/java/com/example/hackathon/domain/user/dto/request/UserNotificationRequest.java, src/main/java/com/example/hackathon/domain/user/dto/response/UserResponse.java
알림 활성화 여부를 검증하는 요청 DTO와 사용자 식별자·닉네임·이메일·이메일 알림 상태를 담는 응답 DTO를 추가했다.
사용자 서비스 및 알림 상태 갱신
src/main/java/com/example/hackathon/domain/user/service/UserService.java, src/main/java/com/example/hackathon/domain/user/entity/User.java
사용자 조회와 존재 검증을 추가하고, 알림 설정 변경 시 사용자 엔티티의 이메일 알림 상태를 갱신한다.
사용자 엔드포인트 및 서비스 테스트
src/main/java/com/example/hackathon/domain/user/controller/UserController.java, src/test/java/com/example/hackathon/domain/user/service/UserServiceTest.java
사용자 조회 GET 엔드포인트와 알림 설정 PATCH 엔드포인트를 추가하고, 두 서비스 동작을 검증하는 테스트를 추가했다.

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

Possibly related PRs

  • 8th-COKERTHON/server-team1#2: 현재 엔드포인트에서 사용하는 공통 응답 래퍼와 사용자 미존재 예외 처리 체계와 연결된다.
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning 제목은 onboarding 복구로 보이지만 실제 변경은 사용자 조회와 알림 설정 API 추가입니다. 변경의 핵심인 사용자 조회 및 알림 설정 기능 추가를 반영해 제목을 구체적으로 수정하세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
✨ 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 feature/onboarding

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

@hyeonszz
hyeonszz merged commit 49e5560 into main Jul 10, 2026
2 checks passed
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