feat/#84-crew: 게시글·댓글 및 무무 도메인 이관 - #85
Open
huncozyboy wants to merge 5 commits into
Open
Conversation
4 tasks
|
Important Review skippedToo many files! This PR contains 133 files, which is 33 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (133)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue 🚀
Work Description ✏️
crew-post-api-controller
GET/post/v2GET/post/v2/{postId}POST/post/v2PUT/post/v2/{postId}DELETE/post/v2/{postId}POST/post/v2/{postId}/likePOST/post/v2/{postId}/reportPOST/post/v2/{postId}/viewcrew-comment-api-controller
GET/comment/v2POST/comment/v2PUT/comment/v2/{commentId}DELETE/comment/v2/{commentId}POST/comment/v2/{commentId}/likePOST/comment/v2/{commentId}/reportadmin-crew-mumu-api-controller
GET/admin/crew/mumu/textPUT/admin/crew/mumu/textPR Point 📸
기존 Crew 서버에 남아있던 게시글 / 댓글 / 무무 관련 기능을 통합 레포 구조에 맞게 이관해주었어요
게시글 / 댓글의 CRUD, 좋아요, 신고 등 공통적인 기능은
domain-playground의 Post 도메인에서 관리하도록 구성했습니다다만 모임 게시글의 조회 / 작성 가능 여부와 같은 Crew 고유의 모임 정책은
domain-crew가 계속 관리하도록 구성했습니다MeetingPostAccessPort를 통해 필요한 모임 접근 정책만 확인하도록 분리해주었어요무무의 경우 일반 게시글과 동일한 정책으로 관리하기 어려운 운영성 기능이기 때문에, 게시글 자체는 공통 Post 도메인을 사용하되 무무 텍스트와 작성 이력 등의 정책은 Crew에 유지했습니다
MumuTextMumuPostWriteHistoryMumuPostPolicyPort기존 Playground에서 Crew 서버의 Internal API를 통해 조회하던 모임 관련 기능들도 통합 서버 구조에 맞게 내부 Port / Adapter 기반으로 변경해주었어요
게시글 / 댓글의 멘션 및 관련 알림은 Post 도메인에서 알림 구현체를 직접 참조하지 않고
PostNotificationSenderPort를 통해 발송하도록 구성했습니다관련 Storage 영역도 함께 이관하여 Post / Comment / Like / Report와 무무 관련 Entity, Repository, Adapter를 통합 레포 구조에 맞게 추가해주었어요