-
Notifications
You must be signed in to change notification settings - Fork 2
🚀🚀🚀🚀 Basic functionality for collaborative editing #102
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
Conversation
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🟢 | Statements | 100% | 0/0 |
| 🟢 | Branches | 100% | 0/0 |
| 🟢 | Functions | 100% | 0/0 |
| 🟢 | Lines | 100% | 0/0 |
Test suite run success
1 tests passing in 1 suite.
Report generated by 🧪jest coverage report action from 909766f
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🟢 | Statements | 99.76% (+3.78% 🔼) |
815/817 |
| 🟢 | Branches | 98.65% (+2.65% 🔼) |
220/223 |
| 🟢 | Functions | 98.5% (+10.83% 🔼) |
197/200 |
| 🟢 | Lines | 99.75% (+3.91% 🔼) |
785/787 |
Test suite run success
418 tests passing in 25 suites.
Report generated by 🧪jest coverage report action from 909766f
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🟡 | Statements | 68% | 17/25 |
| 🔴 | Branches | 20% | 1/5 |
| 🟢 | Functions | 83.33% | 5/6 |
| 🟡 | Lines | 65.22% | 15/23 |
Test suite run success
3 tests passing in 1 suite.
Report generated by 🧪jest coverage report action from 909766f
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🟡 | Statements | 68.35% (-31.65% 🔻) |
162/237 |
| 🟢 | Branches | 81.33% (-16.23% 🔻) |
61/75 |
| 🔴 | Functions | 58.82% (-34.03% 🔻) |
20/34 |
| 🟡 | Lines | 68.38% (-31.62% 🔻) |
160/234 |
Show new covered files 🐣
St.❔ |
File | Statements | Branches | Functions | Lines |
|---|---|---|---|---|---|
| 🟢 | ... / MessageType.ts |
100% | 100% | 100% | 100% |
| 🔴 | client/OTClient.ts | 0% | 0% | 0% | 0% |
| 🟢 | CollaborationManager.ts | 86.76% | 78.26% | 71.43% | 86.76% |
Show files with reduced coverage 🔻
St.❔ |
File | Statements | Branches | Functions | Lines |
|---|---|---|---|---|---|
| 🟢 | Operation.ts | 96% (-4% 🔻) |
96% (-4% 🔻) |
83.33% (-16.67% 🔻) |
96% (-4% 🔻) |
Test suite run success
67 tests passing in 4 suites.
Report generated by 🧪jest coverage report action from 909766f
|
⏭️ No files to mutate for |
|
⏭️ No files to mutate for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 52 out of 52 changed files in this pull request and generated 2 comments.
packages/core/src/entities/index.ts
Outdated
| export type * from './Config.js'; | ||
| export type * from './UnifiedToolConfig.js'; | ||
| export type * from './EditorjsPlugin.js'; |
Copilot
AI
Apr 9, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The usage of 'export type *' is not valid TypeScript syntax. Consider using 'export * from' or the appropriate named type exports instead.
| export type * from './Config.js'; | |
| export type * from './UnifiedToolConfig.js'; | |
| export type * from './EditorjsPlugin.js'; | |
| export * from './Config.js'; | |
| export * from './UnifiedToolConfig.js'; | |
| export * from './EditorjsPlugin.js'; |
| if (message.payload.userId === this.#userId) { | ||
| return; | ||
| } | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
transform undo/redo stack on remote operations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could be done in a separate pr
packages/ui/vite.config.ts.timestamp-1744233098859-624e58abb13.mjs
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <[email protected]>
Co-authored-by: e11sy <[email protected]>
Co-authored-by: Peter <[email protected]>
…re/collaborative-editing
No description provided.