Skip to content

feat: po response#49

Merged
vivivim merged 1 commit intomainfrom
SPM-468
Nov 9, 2025
Merged

feat: po response#49
vivivim merged 1 commit intomainfrom
SPM-468

Conversation

@vivivim
Copy link
Contributor

@vivivim vivivim commented Nov 9, 2025

📝 Summary

에 파트 아이디 추가

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 구매 주문 검색 결과에 부품 ID 정보가 추가되어 더 자세한 주문 정보 조회 가능

에 파트 아이디 추가
@coderabbitai
Copy link

coderabbitai bot commented Nov 9, 2025

Walkthrough

POResDto에 새로운 partId 필드를 추가하고, 이에 따라 @QueryProjection 생성자를 업데이트했습니다. PurchaseOrderQueryRepositoryImpl에서 쿼리 프로젝션에 part.id를 포함하여 새로운 필드를 매핑합니다.

Changes

Cohort / File(s) Summary
DTO 필드 추가
src/main/java/com/sampoom/backend/api/order/dto/POResDto.java
partId 필드(Long) 추가 및 @QueryProjection 생성자에 partId 파라미터 추가
쿼리 프로젝션 업데이트
src/main/java/com/sampoom/backend/api/order/repository/PurchaseOrderQueryRepositoryImpl.java
QPOResDto 생성자 호출 시 part.id 프로젝션 추가 (두 번째 필드 위치)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • DTO 필드 추가와 쿼리 프로젝션 변경이 일관된 패턴을 따릅니다
  • 생성자 파라미터 순서와 프로젝션 매핑이 올바르게 정렬되었는지 확인 필요

Possibly related PRs

  • Spm 442 #44: POResDto와 이 DTO의 생성자/프로젝션 사용법을 동일하게 수정하므로 관련성이 높습니다.

Suggested labels

ready-to-merge

Suggested reviewers

  • CHOOSLA
  • Lee-Jong-Jin
  • yangjiseonn
  • taemin3
  • Sangyoon98

Poem

🐰 새로운 필드 partId가 나타나고,
쿼리는 더 풍부한 데이터를 품으니,
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 제목 'feat: po response'는 변경사항의 핵심인 파트 ID 추가를 명확하게 설명하지 못하고 있습니다. 제목을 'feat: add partId to PO response' 또는 'feat: include part ID in purchase order response'와 같이 구체적으로 변경하여 주요 변경사항을 명확하게 표현하시기 바랍니다.
✅ 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 (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch SPM-468

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d6a0590 and c7fa070.

📒 Files selected for processing (2)
  • src/main/java/com/sampoom/backend/api/order/dto/POResDto.java (3 hunks)
  • src/main/java/com/sampoom/backend/api/order/repository/PurchaseOrderQueryRepositoryImpl.java (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Analyze (java-kotlin)
🔇 Additional comments (2)
src/main/java/com/sampoom/backend/api/order/dto/POResDto.java (1)

18-18: 변경사항이 정확하게 구현되었습니다.

partId 필드가 올바르게 추가되고, @QueryProjection 생성자에 적절히 포함되었습니다. 리포지토리의 쿼리 프로젝션과 일관성 있게 매핑됩니다.

Also applies to: 33-33, 47-47

src/main/java/com/sampoom/backend/api/order/repository/PurchaseOrderQueryRepositoryImpl.java (1)

60-60: 쿼리 프로젝션이 올바르게 업데이트되었습니다.

part.idQPOResDto 생성자의 두 번째 파라미터로 정확하게 추가되었으며, DTO 생성자 시그니처와 완벽하게 일치합니다.


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.

Copy link

@yangjiseonn yangjiseonn left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@vivivim vivivim merged commit b9fb76e into main Nov 9, 2025
7 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