Skip to content

fix: Ensure to fallback to old editor properly on 25 #984

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

Merged
merged 1 commit into from
Mar 22, 2023

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Mar 21, 2023

Fixes #982

I was missing the case where the text is enabled but still on 25 meaning that createEditor is not available.

@juliusknorr juliusknorr added the bug Something isn't working label Mar 21, 2023
@juliusknorr juliusknorr force-pushed the bugfix/noid/editor-25 branch from 5b729fd to 9118e34 Compare March 21, 2023 16:11
@juliusknorr juliusknorr added this to the 4.7.1 milestone Mar 21, 2023
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Just minor comments, otherwise code looks good to me. Didn't test though.

@juliusknorr juliusknorr force-pushed the bugfix/noid/editor-25 branch from 9118e34 to 270c39f Compare March 22, 2023 07:15
@juliusknorr juliusknorr merged commit fb59e26 into master Mar 22, 2023
@juliusknorr juliusknorr deleted the bugfix/noid/editor-25 branch March 22, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Newly integrated rich text editor does not load
2 participants