Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

하위 트리 삭제했는데 커넥션 남아있는 경우 저장되는 문제 해결 #4

Closed
wants to merge 4 commits into from

Conversation

fru1tworld
Copy link
Collaborator

✏️ 한 줄 설명

context tree에서 하위 트리를 삭제했을 때 하위 노드에 connection이 연결된 경우 해당 connection이 종료된 경우 스페이스가 저장되는 문제 해결

✅ 작업 내용

  • update 시점에 존재하는지 여부 확인
  • 추가로 spaceService, noteService 에서 api 서버와 websocket 서버에서 사용되지 않은 로직들 삭제

🏷️ 관련 이슈

📸 스크린샷/영상

📌 리뷰 진행 시 참고 사항

  • API Server는 space, note service가 있는게 자연스럽지만
  • WebSocket Server는 space, note service가 있는게 중복되는 부분이 있습니다.
  • 이 부분은 collaborative service가 space/note service를 의존하고 있어서 생기는 문제인데요.
  • 사실 WebSocket Server의 경우 읽기/저장/업데이트 로직만 수행하면 되므로
  • spaceService랑 noteService가 아닌 collaborativeService에서 하위 model을 생성하는게 자연스럽다고 생각하는데 관련해서 의견이 궁금합니다.

@fru1tworld fru1tworld requested a review from heegenie January 19, 2025 09:44
@fru1tworld fru1tworld closed this Jan 22, 2025
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.

1 participant