Skip to content
This repository was archived by the owner on Jan 11, 2026. It is now read-only.

[REFACTOR] 게시글, 알림, 신고 도메인 모델 구축#13

Merged
msk226 merged 24 commits into
developfrom
SPOT-282/refactor
May 8, 2025
Merged

[REFACTOR] 게시글, 알림, 신고 도메인 모델 구축#13
msk226 merged 24 commits into
developfrom
SPOT-282/refactor

Conversation

@msk226
Copy link
Copy Markdown
Member

@msk226 msk226 commented May 8, 2025

#️⃣ 연관된 이슈


🔎 작업 내용

  1. 도메인 모델 정비
    • 게시글(Post), 알림(Notification), 신고(Report) 도메인 모델 구축 완료
    • PostReportMemberReportreport 도메인으로 통합 예정
  2. 공통 코드 이동
    • S3, 메일, 관리자(Admin), 유틸 관련 Application Layer 코드를 common 패키지 하위로 이동
    • 공통 애노테이션 기반 Validator들도 common 패키지 하위로 이동
  3. 스터디 도메인 정리
    • 기존 legacy 패키지에 있던 스터디 검색(search) 관련 Controller를 study 도메인 패키지로 이전
    • ++ 이거 역할 분담이 좀 애매해 보이는데, 리팩토링 하려면 자기가 개발했던 부분은 자기가 하는게 나으려나요...?
  4. 패키지 구조 개편
    • legacy → refactor 이전 작업 완료
    • 더 이상 사용하지 않는 legacy 패키지는 삭제

📷 스크린샷 (선택)

작업한 결과물에 대한 간단한 스크린샷을 올려주세요.


💬리뷰 요구사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

  • 패키지 간 이동이 많아서 IDE에서 확인 하시면 더 편하게 확인 가능 합니다!

msk226 added 24 commits May 8, 2025 15:06
…> refactor 패키지로 이동

- 현재는 Post 패키지 하위에 모두 위치, 추후 분리 예정
@msk226 msk226 requested review from FromKyoung and dvlp-sy May 8, 2025 06:49
@msk226 msk226 self-assigned this May 8, 2025
@msk226 msk226 added the ♻️ refactor Code Refactoring label May 8, 2025
@msk226 msk226 linked an issue May 8, 2025 that may be closed by this pull request
@dvlp-sy
Copy link
Copy Markdown
Collaborator

dvlp-sy commented May 8, 2025

수정하신 것처럼 Report와 Notification 도메인을 따로 관리하는 방식이 바람직한 것 같습니다~!
앗 그리고 저도 다음 리팩토링 진행할 때 aggregate 관련된 패키지명 association으로 수정하려고 합니다. 통일하는 게 나을 것 같아서요 ㅎㅎ

++ search와 관련된 리팩토링을 마커스님께서 담당하시고, 그 밖의 스터디 기능들을 제가 담당하는 방향은 어떨까요?

@msk226
Copy link
Copy Markdown
Member Author

msk226 commented May 8, 2025

수정하신 것처럼 Report와 Notification 도메인을 따로 관리하는 방식이 바람직한 것 같습니다~! 앗 그리고 저도 다음 리팩토링 진행할 때 aggregate 관련된 패키지명 association으로 수정하려고 합니다. 통일하는 게 나을 것 같아서요 ㅎㅎ

++ search와 관련된 리팩토링을 마커스님께서 담당하시고, 그 밖의 스터디 기능들을 제가 담당하는 방향은 어떨까요?

오 좋아요! 그 부분만 따로 분리시켜주시면 시루님 작업 마무리 된 이후에 검색 기능 리팩토링 할게요!

@msk226 msk226 merged commit ca01a43 into develop May 8, 2025
1 check passed
@msk226 msk226 deleted the SPOT-282/refactor branch May 8, 2025 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

♻️ refactor Code Refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 게시글, 알림 도메인 모델 구축

2 participants