fix(#140): 내 캠페인 조회 api에서 brandId 응답 추가#331
Conversation
Summary of ChangesHello @ParkJiYeoung8297, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 '내 캠페인 조회' API의 응답에 brandId를 추가하여 클라이언트가 캠페인과 관련된 브랜드 정보를 더 쉽게 식별할 수 있도록 개선합니다. 이는 데이터 모델, 리포지토리 쿼리, 서비스 로직 및 응답 DTO 전반에 걸쳐 brandId 필드를 통합하는 변경 사항을 포함합니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
이 PR은 내 캠페인 조회 API 응답에 brandId를 추가하는 변경 사항을 담고 있습니다. CollaborationResponse와 CollaborationProjection DTO에 brandId 필드가 추가되었고, CampaignApplyRepository의 쿼리가 brandId를 조회하도록 수정되었습니다. 전반적으로 올바른 방향의 수정이지만, CampaignProposalRepository에서 CollaborationProjection을 생성하는 로직이 누락되어 RECEIVED 및 SENT 타입의 협업에 대해서는 brandId가 null로 반환될 수 있는 문제가 있습니다. 이 부분을 수정해야 합니다.
| .map(p -> new CollaborationResponse( | ||
| p.campaignId(), | ||
| p.proposalId(), | ||
| p.brandId(), |
There was a problem hiding this comment.
이 PR에서 CampaignApplyRepository의 쿼리는 brandId를 포함하도록 올바르게 수정되었습니다. 하지만 CampaignProposalRepository의 findProposalCollaborations와 searchProposalCollaborations 메소드에 대한 변경 사항이 누락된 것 같습니다. 이 메소드들(아마도 CampaignProposalRepositoryCustomImpl에 구현되어 있을 것입니다)도 brandId를 조회하여 CollaborationProjection에 채워주도록 수정해야 합니다. 그렇지 않으면 RECEIVED 및 SENT 타입의 협업에 대해 brandId가 null이 될 것입니다.
Summary
내 캠페인 조회 api에서 brandId 응답 추가
Changes
Type of Change
Related Issues
참고 사항