fix(#392): FE와 BE 연동에서 채팅에 필요한 ID값 조회되도록 수정#395
Conversation
Summary of ChangesHello @Yoonchulchung, 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은 프론트엔드와 백엔드 연동 시 채팅 기능에 필요한 브랜드 소유자의 사용자 ID를 조회하고 전달할 수 있도록 시스템을 개선합니다. 기존에는 브랜드 ID만 제공되었으나, 이제 브랜드 소유자의 사용자 ID를 다양한 응답 DTO에 포함시켜 클라이언트가 채팅을 시작하는 데 필요한 정보를 확보할 수 있도록 데이터 흐름을 확장했습니다. 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은 채팅 기능에 필요한 brandUserId를 응답에 추가하는 수정이네요. 전반적으로 잘 적용되었지만, BrandDetailResponseDto에서 brandUserId의 타입이 다른 DTO와 일관되지 않은 점을 발견했습니다. 이를 Long 타입으로 통일하여 API의 일관성과 유지보수성을 높이는 방향으로 수정을 제안했습니다. 관련하여 두 파일에 코멘트를 남겼습니다.
I am having trouble creating individual review comments. Click here to see my feedback.
src/main/java/com/example/RealMatch/brand/application/service/BrandService.java (109)
BrandDetailResponseDto의 brandUserId 타입을 Long으로 변경하는 제안에 맞춰, 여기서 String.valueOf() 호출을 제거해야 합니다. 이렇게 하면 타입이 일관되게 유지됩니다.
.brandUserId(brand.getUser().getId())
src/main/java/com/example/RealMatch/brand/presentation/dto/response/BrandDetailResponseDto.java (13)
일관성을 위해 brandUserId 필드의 타입을 Long으로 변경하는 것을 제안합니다. 다른 DTO (예: CollaborationProjection, CollaborationResponse)에서는 brandUserId가 Long 타입으로 사용되고 있습니다. API 전체에서 동일한 데이터 유형을 사용하면 혼동을 줄이고 유지보수성을 향상시킬 수 있습니다.
private Long brandUserId;
Summary
FE와 BE 연동에서 채팅에 필요한 ID값 조회되도록 수정
Changes
Type of Change
Related Issues
#392