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

Feature/BE/#115 server socket.io로 mongo db연동 #116

Merged
merged 18 commits into from
Nov 14, 2024

Conversation

hyonun321
Copy link
Collaborator

📝 변경 사항

image

  • nestJS CRDT 기능 모듈 생성
  • mongoDB 테스트용 document 스키마 생성

🔍 변경 사항 설명

  • socket.io를 통해 클라이언트와 통신한뒤, 클라이언트에 numeric한 번호 부여
  • 연결되면 client에 mongoDB에 있는 data를 문자열로 변환하여 object로 클라이언트에 전달
  • 동시에 문자열로 변환하면서 server의 CRDT에 local.insert를 통해 서버의 CRDT상태도 동기화
  • 클라이언트로부터 연산을 수신하면, server의 CRDT에 반영 후, 나머지 클라이언트에게 broadcast

🙏 질문 사항

  • .env 같은경우 정우님이 말씀해 주신게 있습니다만, 저는 아래와같이 세팅하고 테스트진행했습니다.
MONGO_URI=mongodb://localhost:27017/test
PORT=3000
HOST=0.0.0.0

📷 스크린샷 (선택)

image

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@hyonun321 hyonun321 added BE 백엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업 labels Nov 14, 2024
@hyonun321 hyonun321 self-assigned this Nov 14, 2024
Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

정말 고생하셨습니다...

Copy link
Collaborator

@minjungw00 minjungw00 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!

@github-actions github-actions bot merged commit 093f167 into dev Nov 14, 2024
5 checks passed
@hyonun321 hyonun321 deleted the Feature/BE/#115_server-socket.io로_mongoDB연동 branch November 15, 2024 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 작업 Feat 새로운 기능 추가나 기존 기능 확장 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants