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/#122 mongo db block crdt연동 #124

Merged
merged 7 commits into from
Nov 14, 2024

Conversation

hyonun321
Copy link
Collaborator

📝 변경 사항

image

  • MongoDB에 저장되는 doc 의 스키마를 DB스키마에 따라 JSON구조로 생성하게 했습니다.

🔍 변경 사항 설명

  • 이에따라 serverice, gateway에서 직렬화/역직렬화 및 localInsert, localDelete등의 메소드가 수정되었습니다.

image

  • mongoDB에 스키마에 따라 doc을 생성합니다.

번외

  async getDocument(): Promise<Doc> {
    let doc = await this.documentModel.findOne();
    if (!doc) {
      doc = new this.documentModel({ content: JSON.stringify(this.crdt.spread()) });
      await doc.save();
    }
    return doc;
  }
  • 아직 모델이 없는경우는 content로 지정되는 경우가 있어 해당 부분은 EditorCRDT, Page, WorkSpace등으로 확장할때 수정할 예정입니다.

🙏 질문 사항

  • 추후 JEST로 동작 테스트 여부가 잘될지 궁금합니다..!

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

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

hyonun321 and others added 7 commits November 15, 2024 01:56
- 불필요한 주석 제거
- BlockCRDT구조에 따라 역직렬화 코드 추가

#122
- crdt에서 일어나는 건 crdt폴더 내에서만 일어나도록 feature화

#122
https -> http
…into Feature/#122_mongoDB_BlockCRDT연동
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.

LGTM

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 3ab5d52 into dev Nov 14, 2024
5 checks passed
@hyonun321 hyonun321 deleted the Feature/#122_mongoDB_BlockCRDT연동 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 백엔드 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants