Skip to content

Conversation

@leesumin0526
Copy link
Collaborator

@leesumin0526 leesumin0526 commented Sep 25, 2025

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 리팩터링

반영 브랜치

dev -> main

작업 내용

/user/profile/update API의 requestBody에서 kakaoId 제거

테스트 결과

스크린샷 2025-09-25 오후 4 35 01

Summary by CodeRabbit

  • 신규 기능: 해당 없음
  • 변경 사항
    • 사용자 프로필 업데이트에서 카카오 아이디(kakaoId) 항목을 제거하여 더 이상 수정할 수 없습니다.
  • 문서
    • 사용자 업데이트 API 스키마/문서에서 kakaoId 항목을 제거했습니다.

- requestBody에서 kakaoId 제거
@coderabbitai
Copy link

coderabbitai bot commented Sep 25, 2025

Walkthrough

사용자 프로필 업데이트 기능에서 kakaoId 필드와 관련 로직을 제거했습니다. 요청 DTO에서 kakaoId 속성을 삭제했고, 서비스의 업데이트 메서드에서 kakaoId를 설정하는 분기 처리를 제거했습니다.

Changes

Cohort / File(s) Change Summary
DTO 요청 스키마 정리
src/main/java/stackpot/stackpot/user/dto/request/UserUpdateRequestDto.java
kakaoId 필드와 해당 @Schema 주석 제거. 직렬화/역직렬화 및 공개 API 스키마에서 kakaoId 제외.
서비스 업데이트 로직 단순화
src/main/java/stackpot/stackpot/user/service/UserCommandServiceImpl.java
updateUserProfile 내 요청으로부터 kakaoId를 설정하던 분기 삭제. 다른 필드/로직 변경 없음.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor Client
    participant API as User API
    participant Service as UserCommandServiceImpl
    participant Repo as UserRepository

    Client->>API: PUT /users/{id} (프로필 업데이트 요청)
    API->>Service: updateUserProfile(request without kakaoId)
    rect rgba(200,230,255,0.3)
      note over Service: kakaoId 관련 처리 단계 제거
      Service->>Repo: 사용자 조회 및 업데이트(기존 필드만)
      Repo-->>Service: 저장 결과
    end
    Service-->>API: 업데이트 결과 반환
    API-->>Client: 200 OK
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

:fire: fix

Poem

귀가 번쩍, 코드 톡톡—봄바람 같네
카카오는 내려놓고 가벼워졌네 🐇
DTO도, 서비스도 한결 단정—퐁당
필드 한 잎 떨궜더니 흐름이 맑다
휘릭–커밋! 오늘도 당근 한 입 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title Check ❓ Inconclusive PR 제목인 "[Merge/#457] dev -> main 병합"은 브랜치 병합만을 언급하며 핵심 변경인 kakaoId 제거를 반영하지 않아 변경 내용을 명확히 전달하지 못합니다. 제목을 "사용자 프로필 업데이트 요청에서 kakaoId 필드 제거"와 같이 핵심 변경 사항을 구체적으로 요약하도록 수정해 주세요.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 623634b and 967375b.

📒 Files selected for processing (2)
  • src/main/java/stackpot/stackpot/user/dto/request/UserUpdateRequestDto.java (0 hunks)
  • src/main/java/stackpot/stackpot/user/service/UserCommandServiceImpl.java (0 hunks)
💤 Files with no reviewable changes (2)
  • src/main/java/stackpot/stackpot/user/service/UserCommandServiceImpl.java
  • src/main/java/stackpot/stackpot/user/dto/request/UserUpdateRequestDto.java

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.

@leesumin0526 leesumin0526 changed the title [FEAT/#457] (#460) [Merge/#457] dev -> main 병합 Sep 25, 2025
@leesumin0526 leesumin0526 self-assigned this Sep 25, 2025
@leesumin0526 leesumin0526 merged commit f90a65c into main Sep 25, 2025
2 checks passed
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