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/#121 crdt 라이브러리 db스키마로 개선 #123

Merged

Conversation

hyonun321
Copy link
Collaborator

📝 변경 사항

image

  • CRDT라이브러리를 DB스키마에 맞게 활용할 수 있도록 개선합니다.

🔍 변경 사항 설명

  • CRDT -> EditorCRDT, BlockCRDT 클래스 상속하여 생성
  • LinkedList -> BlockLinkedList, TextLinkedList 클래스 상속하여 생성
  • Node, NodeID 등 Block과 Char에 따라 Id 클래스 생성함

🙏 질문 사항

  • 추후 client와 server에서 직렬화/역직렬화를 할 예정이니, 구조 확인부탁드립니다!

📷 스크린샷 (선택)

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

✅ 작성자 체크리스트

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

- 불필요한 주석 제거
- EditorCRDT, BlockCRDT 클래스 생성
- CRDT에 제네릭 타입 추가하여 확장성 부여

#121
- 추후 사용가능성을 위해 주석처리

#121
- BlockLinkedList, TextLinkedList 생성
- 링크드리스트에 제네릭 선언하여 확장성 부여

#121
- Block, Char 클래스 생성
- Node 에 제네릭 타입 선언하여 확장성 추가

#121
- 각 클래스에 사용되는 클래스들 재사용

#121
@hyonun321 hyonun321 added the Feat 새로운 기능 추가나 기존 기능 확장 작업 label Nov 14, 2024
@hyonun321 hyonun321 self-assigned this Nov 14, 2024
@hyonun321
Copy link
Collaborator Author

CRDT 라이브러리 이전의 요소를 server에서 보고있어서 발생하는 에러입니다.
해당 에러는 추후 server쪽 코드를 업데이트하면 에러가 발생하지 않을 예정입니다.
먼저 Merge 하겠습니다.

@hyonun321 hyonun321 merged commit be122c0 into dev Nov 14, 2024
0 of 3 checks passed
@hyonun321 hyonun321 deleted the Feature/#121_CRDT_라이브러리_DB스키마로_개선 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
Feat 새로운 기능 추가나 기존 기능 확장 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant