payload-plugin-lexical icon indicating copy to clipboard operation
payload-plugin-lexical copied to clipboard

update lexical to 0.11.1

Open timofei-iatsenko opened this issue 2 years ago • 4 comments

I tried to update it locally via yarn's resolutions, but it seems there are breaking changes between current and latest version.

TypeError: Class extends value undefined is not a constructor or null at Object. (/Users/tim/projects/roadtravel-main-landing/backend/node_modules/payload-plugin-lexical/src/features/emojipicker/plugins/index.tsx:26:27)

timofei-iatsenko avatar Jun 20 '23 13:06 timofei-iatsenko

I tried to update it locally via yarn's resolutions, but it seems there are breaking changes between current and latest version.

TypeError: Class extends value undefined is not a constructor or null at Object. (/Users/tim/projects/roadtravel-main-landing/backend/node_modules/payload-plugin-lexical/src/features/emojipicker/plugins/index.tsx:26:27)

oh yea, updating needs some extra work. Usually, I go through each commit of the lexical playground and copy the relevant stuff they changed there.

Will do that once I find time! Don't think there have been any too exciting changes anyways

AlessioGr avatar Jun 20 '23 13:06 AlessioGr

I personally encountered a problem on our project. When a document has tables inside in some circumstances copy/paste stop working. When you trying to copy it show some not very useful error in console and all further processing is stopped. So i decided to check on new version first and that report a bug if it doesn't fixed in upstream.

timofei-iatsenko avatar Jun 20 '23 15:06 timofei-iatsenko

I personally encountered a problem on our project. When a document has tables inside in some circumstances copy/paste stop working. When you trying to copy it show some not very useful error in console and all further processing is stopped. So i decided to check on new version first and that report a bug if it doesn't fixed in upstream.

Yep, that's https://github.com/AlessioGr/payload-plugin-lexical/issues/5. Already reported the bug upstream - sadly it has not been fixed in the latest version yet.

There's an open PR which fixes the issue, also linked in the upstream issue, but it's currently stalled. Might have to check it out and finish it myself

AlessioGr avatar Jun 20 '23 15:06 AlessioGr

i saw this issue, but i think it's a different one. Firstly i have only one editor on the page, secondly the error message is different.

timofei-iatsenko avatar Jun 20 '23 15:06 timofei-iatsenko