Conversation
…y 디렉토리 위치 수정(auth -> member)
Summary of ChangesHello @yeongsinkeem, 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! 이 Pull Request는 운영진이 동아리/소모임 회원을 관리하고 신고된 게시글을 처리할 수 있는 핵심 기능을 도입합니다. 또한, 시스템 전반에 걸쳐 계정 및 게시글의 활성 상태를 나타내는 Enum을 'AccountStatus'에서 'ActiveStatus'로 리팩토링하여 코드의 일관성과 가독성을 향상시켰습니다. 이러한 변경사항은 관리자 및 운영진의 효율적인 플랫폼 운영을 지원하며, 데이터 모델의 정확성을 높입니다. Highlights
Changelog
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
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 운영진을 위한 회원 관리 API를 추가하고, 전반적인 코드베이스에서 AccountStatus를 ActiveStatus로 리팩토링하는 등 많은 변경 사항을 포함하고 있습니다. 새로운 기능 추가와 함께 코드의 명확성을 높이는 리팩토링이 이루어진 점이 좋습니다. 몇 가지 개선점을 제안합니다. Report 엔티티 생성 시 reportedAt 필드가 누락되어 있어 오류가 발생할 수 있습니다. 또한, 새로 추가된 ExecutiveMemberService의 회원 조회 로직에 "ALL" 상태를 처리하는 부분에 버그가 있습니다. 마지막으로, 새로 추가된 ReportPost 엔티티가 기존 Report 엔티티와 기능이 중복되어 보여 코드 중복 및 혼란을 야기할 수 있어 검토가 필요해 보입니다. 자세한 내용은 각 파일의 리뷰 코멘트를 참고해주세요.
src/main/java/org/example/tackit/domain/executive/service/ExecutiveMemberService.java
Outdated
Show resolved
Hide resolved
| @Entity | ||
| @Getter | ||
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | ||
| @Table(name = "report_post") | ||
| public class ReportPost { |
There was a problem hiding this comment.
새로 추가된 ReportPost 엔티티가 리팩토링된 Report 엔티티와 매우 유사하여 기능이 중복되는 것으로 보입니다. Report 엔티티는 이미 targetType과 postType 필드를 통해 다양한 종류의 신고(게시글, 댓글)를 처리할 수 있습니다. 또한, ReportPost 엔티티는 현재 코드베이스에서 사용되지 않고 있습니다. 코드 중복과 혼란을 피하기 위해 Report 엔티티로 기능을 통합하고 ReportPost 엔티티를 제거하는 것을 고려해 보세요.
There was a problem hiding this comment.
게시판 기능 통합한 후 신고 게시글도 통합하여 리팩토링 예정
| @Column(nullable = false) | ||
| private ReportReason reportReason; | ||
|
|
||
| @Column(nullable = false) |
There was a problem hiding this comment.
@yeongsinkeem
저같은 경우에는 이 경우에 builder에서 따로 localdatetime.now()를 넣는 대신에 @CreatedDate 어노테이션을 넣는 형태로 event, poll 객체 생성했었는데 추후에 통일하는 게 좋을 것 같습니다. 일단 당장은 메인 배포를 해야 프론트엔드에서 맞춰볼 수 있을 것 같아서 머지는 하겠습니다!
2 participants
이슈 번호
작업 내용