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

feat: Refactor yjs editor #8253

Merged
merged 5 commits into from
Nov 24, 2023
Merged

Conversation

jam411
Copy link
Contributor

@jam411 jam411 commented Nov 23, 2023

task: https://redmine.weseek.co.jp/issues/132901

内容

リファクタリングを進めました。もっと良くなるとは思いますが、前回 FB 分 + 少し の修正をしました。

  • yjs-connection-manager のリネーム
  • useGlobalSocket 等 yjs に必要な socket 関連を @growi/core へ移動
  • yjs コードの hooks 化

今後

undo や、リビジョン参照など、機能の実装を優先しようと思っています。
https://redmine.weseek.co.jp/issues/132778
https://redmine.weseek.co.jp/issues/134397

@yuki-takei yuki-takei merged commit 7530988 into feat/yjs-editor Nov 24, 2023
@yuki-takei yuki-takei deleted the feat/132901-refactor-yjs-editor branch November 24, 2023 01:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants