Skip to content

Conversation

@fru1tworld
Copy link
Member

✏️ 한 줄 설명

API서버와 WebSocket을 분리함에 따라 NoteService와 SpaceService를 의존해서 불필요한 모듈에 의존하던 문제 CollaborativeService에서 NoteService와 SpaceService를 의존하는것이 아니라 CollaborativeSpaceService, CollaborativeNoteService로 분리하여 의존도를 낮췄습니다.

✅ 작업 내용

  • NoteService 및 SpaceService 삭제

📸 스크린샷/영상

📌 리뷰 진행 시 참고 사항

@fru1tworld fru1tworld requested a review from heeyozip January 22, 2025 11:36
@fru1tworld fru1tworld closed this Jan 22, 2025
Copy link
Collaborator

@heeyozip heeyozip left a comment

Choose a reason for hiding this comment

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

기존에 노트와 스페이스 별로 분리되어 있던 로직들을 동시편집 디렉토리 하위에서 관리하는 걸로 구조를 변경하신 것 같네요! 고생 많으셨습니다 🙌

providers: [CollaborativeService],
exports: [CollaborativeService],
imports: [
MongooseModule.forFeature([
Copy link
Collaborator

@heeyozip heeyozip Jan 22, 2025

Choose a reason for hiding this comment

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

기존에 그냥 import하는 것과 MongooseModule을 활용해서 import하는 것엔 어떤 차이가 있나요?

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.

3 participants