Skip to content

Conversation

@aonnikov
Copy link
Member

No description provided.

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
@aonnikov aonnikov requested review from Copilot and haiodo October 13, 2025 06:37
@huly-github-staging
Copy link

Connected to Huly®: UBERF-13917

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the IndexedDB provider implementation from the text editor resources, simplifying the collaborative text editor to use only remote providers for synchronization.

  • Removed the entire IndexedDB provider implementation and its usage
  • Simplified the collaborative text editor to use a single provider instead of dual local/remote providers
  • Cleaned up package dependencies by removing unused libraries

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
plugins/text-editor-resources/src/provider/indexeddb.ts Completely removed the IndexedDB provider implementation
plugins/text-editor-resources/src/provider/utils.ts Removed createLocalProvider function and IndexeddbProvider import
plugins/text-editor-resources/src/components/CollaborativeTextEditor.svelte Simplified to use single provider, removed local provider logic and state tracking
plugins/text-editor-resources/package.json Removed unused dependencies y-websocket and y-indexeddb

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

haiodo
haiodo previously approved these changes Oct 13, 2025
Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
@haiodo haiodo merged commit 704e619 into develop Oct 13, 2025
25 of 26 checks passed
@haiodo haiodo deleted the fix-remove-indexeddb branch October 13, 2025 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants