[FIX] postId는 postID, leaderId는 leaderId를 반환하도록 함

[88guri]

📌 관련 이슈

• close [FIX] leaderId 반환 대신 postId 반환중 #187

✨ 변경 사항

• leaderUserID가 postId를 반환하고있어서 수정했습니다

📸 테스트 증명 (필수)

📚 리뷰어 참고 사항

✅ 체크리스트

• 브랜치 전략(git flow)을 따랐나요? (develop -> feat/...)
• 로컬에서 빌드 및 실행이 정상적으로 되나요?
• 불필요한 주석(TODO 등)이나 더미 코드는 제거했나요?
• 컨벤션(커밋 메시지, 코드 스타일)을 지켰나요?

Summary by CodeRabbit

릴리스 노트

• 개선 사항
  • 배송 확인 응답에 그룹 구매 게시물 리더 정보가 포함되도록 개선되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

confirmDelivered 메서드에서 그룹 바이 포스트의 리더 ID를 조회하여 LeaderUserIdResponse 반환값에 추가하는 변경사항입니다. 응답 객체의 생성자 시그니처가 확장되어 postId와 leaderUserId를 모두 반환합니다.

Changes

Cohort / File(s)	Summary
응답 DTO 및 서비스 로직 src/main/java/org/sopt/poti/domain/participation/dto/response/LeaderUserIdResponse.java, src/main/java/org/sopt/poti/domain/participation/service/ParticipationService.java	confirmDelivered 메서드에서 order.getGroupBuyPost().getLeader().getId()를 통해 leaderUserId를 조회하여 응답에 포함. LeaderUserIdResponse 생성자 시그니처 변경: (Long postId) → (Long postId, Long leaderUserId)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• [FIX] leaderuserid 반환 #179: confirmDelivered 메서드와 LeaderUserIdResponse 타입을 동시에 수정하는 관련 PR

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 주요 변경사항을 명확히 반영하고 있습니다. postId와 leaderId가 올바르게 반환되도록 수정했다는 내용이 직관적으로 전달됩니다.
Linked Issues check	✅ Passed	PR 변경사항이 #187 이슈의 요구사항을 충족합니다. leaderId 대신 postId를 반환하던 버그를 수정하여 LeaderUserIdResponse에 leaderUserId 필드를 추가했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #187 이슈 해결을 위한 범위 내입니다. LeaderUserIdResponse 생성자 수정과 confirmDelivered 메서드의 leaderUserId 추가 로직만 포함되어 있습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/main/java/org/sopt/poti/domain/participation/service/ParticipationService.java (1)

75-89: LeaderUserIdResponse 생성자 인자 누락으로 컴파일/반환값 오류

현재 new LeaderUserIdResponse(leaderUserId)로 호출하고 있어, 생성자가 (postId, leaderUserId)로 변경된 경우 컴파일 실패 또는 의도한 postId 누락이 발생합니다. PR 목적에 맞게 postId도 함께 반환하도록 수정해주세요.

🐛 수정 제안

-    return new LeaderUserIdResponse(leaderUserId);
+    return new LeaderUserIdResponse(postId, leaderUserId);