ckeditor5 icon indicating copy to clipboard operation
ckeditor5 copied to clipboard

Object literal may only specify known properties, and 'table' does not exist in type 'EditorConfig'.

Open AbineshDorustree opened this issue 1 year ago • 6 comments

  1. …After integration it in angular
  2. …create a custom ckeditor for mentions.

✔️ Expected result

A normal running Angular application.

❌ Actual result

Class static side 'typeof Editor' incorrectly extends base class static side 'typeof ClassicEditor'. The types of 'defaultConfig.context' are incompatible between these types. Type 'import(".../CK editor/queue-management-system/Frontend/node_modules/@ckeditor/ckeditor5-core/src/context").default | undefined' is not assignable to type 'import(".../CK editor/queue-management-system/Frontend/node_modules/ckeditor5/node_modules/@ckeditor/ckeditor5-core/src/context").default | undefined'. Type 'import(".../CK editor/queue-management-system/Frontend/node_modules/@ckeditor/ckeditor5-core/src/context").default' is not assignable to type 'import(".../CK editor/queue-management-system/Frontend/node_modules/ckeditor5/node_modules/@ckeditor/ckeditor5-core/src/context").default'. Types of property 'config' are incompatible. Type 'import(".../CK editor/queue-management-system/Frontend/node_modules/@ckeditor/ckeditor5-utils/src/config").default<import(".../CK editor/queue-management-system/Frontend/node_modules/@ckeditor/ckeditor5-core/src/context").ContextConfig>' is not assignable to type 'import(".../CK editor/queue-management-system/Frontend/node_modules/ckeditor5/node_modules/@ckeditor/ckeditor5-utils/src/config").default<import(".../CK editor/queue-management-system/Frontend/node_modules/ckeditor5/node_modules/@ckeditor/ckeditor5-core/src/context").ContextCo...'. Types have separate declarations of a private property '_config'.

❓ Possible solution

Not yet found

📃 Other details

  • Browser: …
  • OS: …linux
  • First affected CKEditor version: …5
  • Installed CKEditor plugins: …41.0.0 Screenshot from 2024-02-12 09-48-49

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

AbineshDorustree avatar Feb 12 '24 04:02 AbineshDorustree

This was working fine till last month, but after building it with the latest npm it is throwing an error. And there were no code-level changes in it.

AbineshDorustree avatar Feb 12 '24 08:02 AbineshDorustree

Here is the finding on the previous working node_modules and current non-working node_modules

The interface EditorConfig is been mispointed out in the generated node_modules package.

Screenshot from 2024-02-13 16-27-57

Screenshot from 2024-02-13 16-18-28

The working one has been referred from the ckeditor5-core but the latest is been pointed out to the ckeditor5-ui

AbineshDorustree avatar Feb 14 '24 04:02 AbineshDorustree

The new version `41.0.0` provides types with TypeScript 5.0, the screenshots that you provided are of little help TBH. Most likely, we would need some reproducible sample to dig into this. @filipsobol any thoughts?

Witoso avatar Feb 15 '24 08:02 Witoso

The new version 41.0.0 provides types with TypeScript 5.0, the screenshots that you provided are of little help TBH. Most likely, we would need some reproducible sample to dig into this. @filipsobol any thoughts?

@Witoso Thanks for the update, will try to share some sample of this reproducible issue with you in a short while.

AbineshDorustree avatar Feb 15 '24 10:02 AbineshDorustree

I confirm that this issue has been resolved in the new version 41.1.0.

tranphuquy19 avatar Feb 15 '24 18:02 tranphuquy19

I confirm that this issue has been resolved in the new version 41.1.0.

Thanks for the breakthrough, we were using TypeScript 4.9.5 and the CKEditor version 41.0.0.

Once the "ckeditor5-editor-classic" version is been updated to 41.1.0 and the above issues have been resolved.

Thanks a lot, @Witoso @tranphuquy19 Your input has been invaluable throughout this process. I especially appreciate your thorough testing and attention to detail. 🙌

AbineshDorustree avatar Feb 16 '24 05:02 AbineshDorustree

Hmm, I must say, I'm surprised the bump helped, as I cannot find any related work on our side, but I'm glad this is resolved. I'm closing, if anyone in the future will stumble upon this problem in some version, please comment, and we will reopen.

Witoso avatar Feb 26 '24 13:02 Witoso