Skip to content

[REFACTOR] 댓글/알림/탈퇴 흐름 리팩터링 및 공통 로직 정리#541

Merged
ljy1348 merged 20 commits into
devfrom
refactor/#539
Jun 30, 2026
Merged

[REFACTOR] 댓글/알림/탈퇴 흐름 리팩터링 및 공통 로직 정리#541
ljy1348 merged 20 commits into
devfrom
refactor/#539

Conversation

@GiJungPark

Copy link
Copy Markdown
Member

Related Issue

Key Changes

댓글 도메인, 알림 전송, 회원 탈퇴 처리, 공통 엔티티/유틸 정리를 묶어서 정리했습니다.
댓글은 조회/수정/삭제 흐름을 단순화하고, 알림은 이벤트/정책 기반으로 분리했으며, 탈퇴 시 데이터 정리와 공통 베이스 엔티티 사용을 정리했습니다.
또한 FK 구조로 인해 댓글이 삭제되지 않는 버그를 수정했습니다.

변경 상세 내역

  • 댓글 작성 권한 체크를 컨트롤러에서 제거하고 애플리케이션/도메인 책임으로 이동

  • 댓글 수정/삭제 로직을 feed 의존 경로와 분리하고, 단일 댓글 경로를 정리

  • 댓글 조회를 Projection + QueryDSL 기반으로 변경

  • 댓글 관련 불필요한 CommentAuthorizationValidator 제거

  • 댓글 엔티티 정리

    • 필드명/메서드명 정리
    • boolean 기반 상태값으로 정리
    • @DynamicInsert 제거
    • 생성자 순서와 주석 정리
    • BaseEntity 적용으로 생성/수정 시각 공통화
    • 생성/수정 시 댓글 content 유효성 검증 추가
  • 댓글 신고/차단 정책 보완

    • 단방향 차단 시 양쪽 모두 댓글을 볼 수 없도록 수정
  • 댓글 작성 시 알림 전송 로직을 이벤트/리스너/정책으로 분리

  • 회원 탈퇴 시 피드/댓글 작성자 unknown 처리 정리

  • Action enum 제거 및 관련 코드 정리

  • 불필요한 트랜잭션 정리

  • 데이터 베이스 구조에 맞게 삭제 로직을 수정 (삭제가 안되던 버그)

To Reviewers

References

@github-actions github-actions Bot requested review from ljy1348 and sansan20535 June 28, 2026 12:58
@ljy1348 ljy1348 merged commit b1f82ac into dev Jun 30, 2026
2 checks passed
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.

[REFACTOR] 댓글 도메인 결합도 및 삭제 제약 개선

2 participants