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

Fixes the issue where CKE 5 Interface not loading #449

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

vincent-gao
Copy link
Contributor

@vincent-gao vincent-gao commented Oct 29, 2023

Problem/Motivation

https://www.drupal.org/project/entity_embed/issues/3272732 has been merged, but it introduces a new issue to Drupal 9, where CKEditor 5 Interface is not loading on text formats using entity embed.

Fix

https://www.drupal.org/project/entity_embed/issues/3396133
https://git.drupalcode.org/project/entity_embed/-/merge_requests/26/diffs

Copy link
Contributor

@anthony-malkoun anthony-malkoun left a comment

Choose a reason for hiding this comment

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

Is the patch not on drupal.org as well? Why not use that one?

@vincent-gao
Copy link
Contributor Author

Is the patch not on drupal.org as well? Why not use that one?

hey @anthony-malkoun, There is no patch file available on Drupal.org, only a PR. https://git.drupalcode.org/project/entity_embed/-/merge_requests/26

@krakerag
Copy link
Contributor

Having a static patch is better than the PR, which can change. Is our pattern to store a patch on drupal.org or locally? Whichever the chosen pattern is, lets match that.

@vincent-gao
Copy link
Contributor Author

hey @krakerag
The patch is static and stored locally,
https://github.com/dpc-sdp/tide_core/pull/449/files#diff-d2ab9925cad7eac58e0ff4cc0d251a937ecf49e4b6bf57f8b95aab76648a9d34R248

@vincent-gao vincent-gao merged commit 8b85e9a into develop Oct 30, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants