Skip to content

어드민 신고 목록 조회 응답 필드 추가#62

Merged
cfcromn merged 2 commits intodevelopfrom
refactor/report
Dec 27, 2025
Merged

어드민 신고 목록 조회 응답 필드 추가#62
cfcromn merged 2 commits intodevelopfrom
refactor/report

Conversation

@hej090224
Copy link
Collaborator

💡 배경 및 개요

클라이언트의 요청으로 어드민 신고 목록 조회 API 응답의 필드 추가

📃 작업내용

AdminReportResponsereportResult, reporterName, reportedPostId 필드를 추가하였습니다.

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@hej090224 hej090224 requested a review from cfcromn December 27, 2025 13:59
@hej090224 hej090224 self-assigned this Dec 27, 2025
@gemini-code-assist
Copy link

Summary of Changes

Hello @hej090224, 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의 응답 데이터 구조를 확장합니다. 기존 응답에 신고 처리 결과, 신고자 이름, 신고된 게시물 ID와 같은 중요한 정보들을 추가하여, 어드민 페이지에서 신고 내역을 더욱 상세하고 효율적으로 관리할 수 있도록 개선하는 것이 목적입니다.

Highlights

  • 응답 필드 추가: 클라이언트 요청에 따라 어드민 신고 목록 조회 API의 응답에 새로운 필드들이 추가되었습니다.
  • AdminReportResponse DTO 변경: AdminReportResponse DTO에 reportResult, reporterName, reportedPostId 필드가 추가되었습니다.
  • 데이터 매핑 로직 업데이트: AdminReportResponsefrom 팩토리 메서드가 업데이트되어 새로 추가된 필드들을 Report 엔티티로부터 매핑하도록 변경되었습니다.
Using Gemini Code Assist

The 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 /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.

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

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

PR의 내용은 클라이언트 요청에 따라 AdminReportResponsereportResult, reporterName, reportedPostId 필드를 추가하는 것입니다. 변경 사항은 명확하지만, 새로 추가된 필드들이 지연 로딩(lazy-loaded) 관계에 있는 엔티티(reporter, post)에 접근하면서 심각한 성능 문제(N+1 쿼리 문제)를 유발할 수 있습니다. 리뷰에서는 이 문제에 대한 상세한 설명과 성능 개선을 위해 JOIN FETCH를 사용하는 해결 방안을 제안했습니다.

Copy link
Collaborator

@cfcromn cfcromn left a comment

Choose a reason for hiding this comment

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

ㅅㄱㅇ

@cfcromn cfcromn merged commit 4c80367 into develop Dec 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants