Skip to content

Feat/#221 admin report#223

Open
yeongsinkeem wants to merge 9 commits intodevfrom
feat/#221-admin-report
Open

Feat/#221 admin report#223
yeongsinkeem wants to merge 9 commits intodevfrom
feat/#221-admin-report

Conversation

@yeongsinkeem
Copy link
Copy Markdown
Collaborator

이슈 번호

#221

작업 내용

  • 관리자 신고 api : 전체 조회, 상세 조회, 활성화, 완전 삭제

@yeongsinkeem yeongsinkeem requested a review from tishakong April 3, 2026 02:58
@yeongsinkeem yeongsinkeem self-assigned this Apr 3, 2026
Copy link
Copy Markdown
Contributor

@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

This pull request refactors the admin member and post management modules, renaming DTOs for consistency and implementing new functionality for reported posts. Key changes include new controller endpoints, service logic for post activation/deletion, and repository queries for report filtering. Feedback highlights a performance issue with N+1 queries in AdminPostService, duplicate error codes in ErrorCode.java, and the need to remove commented-out code.

Comment thread src/main/java/org/example/tackit/domain/admin/service/AdminPostService.java Outdated
Comment thread src/main/java/org/example/tackit/global/exception/ErrorCode.java Outdated
Comment thread src/main/java/org/example/tackit/domain/admin/service/AdminPostService.java Outdated
Comment thread src/main/java/org/example/tackit/domain/admin/dto/ReportedPostDto.java Outdated

// [ 신고 게시글 상세 조회 ]
@GetMapping("/{reportId}")
public ResponseEntity<ReportedPostDetailDto> getReportedPostDetail(@PathVariable Long reportId) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

저희 토큰? 권한 검사는 어디서 하고 있나요?

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