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

CKEditor error: Cannot read properties of null (reading 'destroy') #913

Closed
sentry-io bot opened this issue Jan 20, 2022 · 2 comments
Closed

CKEditor error: Cannot read properties of null (reading 'destroy') #913

sentry-io bot opened this issue Jan 20, 2022 · 2 comments

Comments

@sentry-io
Copy link

sentry-io bot commented Jan 20, 2022

Sentry Issue: OCW-STUDIO-1

TypeError: Cannot read properties of null (reading 'destroy')
  at _destructor (./node_modules/@ckeditor/ckeditor5-react/dist/ckeditor.js:5:20995)
  at None (./node_modules/@ckeditor/ckeditor5-react/dist/ckeditor.js:5:22255)
@alicewriteswrongs alicewriteswrongs added this to Backlog in OCW Next via automation Jan 20, 2022
@alicewriteswrongs alicewriteswrongs moved this from Backlog to In progress in OCW Next Jan 20, 2022
@alicewriteswrongs
Copy link
Contributor

it looks like this is an issue with the @ckeditor/ckeditor5-react package, there's an issue on the repo about it here: ckeditor/ckeditor5-react#241

After reading through the relevant code on that repo and reproducing the error a bunch locally I think we can safely ignore it for now on sentry because it does not, as far as I can tell, have any user-facing impact. The editor still works just fine without any problems - the error is basically happening once in the first refresh that happens (as far as I can tell) and not afterwards.

So I'll silence the issue in Sentry and then follow the relevant issue on the upstream repo, so we can update if / when they fix it.

@pdpinch pdpinch moved this from In progress to Backlog in OCW Next Feb 17, 2022
@alicewriteswrongs alicewriteswrongs removed their assignment Mar 22, 2022
@pdpinch pdpinch removed this from Backlog in OCW Next Apr 1, 2022
@pdpinch
Copy link
Member

pdpinch commented May 2, 2023

we decided to ignore this error

@pdpinch pdpinch closed this as completed May 2, 2023
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

No branches or pull requests

2 participants