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/#117 crdt 라이브러리 개선 #118

Merged
merged 4 commits into from
Nov 14, 2024

Conversation

hyonun321
Copy link
Collaborator

📝 변경 사항

  • CRDT 라이브러리 내 WorkSpace, Page 클래스 생성
  • LinkedList, CRDT 내부 객체 및 메소드 이름 변경

🔍 변경 사항 설명

image

  • DB스키마에 맞게 필요한 클래스들을 생성하였습니다.

🙏 질문 사항

  • 추후 client 및 server와 연동하며 내부 동작을 개선할 예정입니다.

📷 스크린샷 (선택)

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

✅ 작성자 체크리스트

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

@hyonun321 hyonun321 added the Refactor 기존 코드의 구조나 성능을 개선 및 코드 가독성, 유지보수성 작업 label 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.

LGTM

@hyonun321
Copy link
Collaborator Author

현재 CRDT구조를 dev에 적용하고, 안정적인 동작이 되는 dev환경에서 기존 프로토타입 socket형식의 testApp.tsx에서 CRDT동작을 점검 해보기위해 먼저 Merge 하겠습니다!
@pipisebastian @minjungw00

@hyonun321 hyonun321 merged commit f2744c1 into dev Nov 14, 2024
4 checks passed
@hyonun321 hyonun321 deleted the Feature/#117_CRDT_라이브러리_개선 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
Refactor 기존 코드의 구조나 성능을 개선 및 코드 가독성, 유지보수성 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants