Closed (fixed)
Project:
Paragraphs Inline Entity Form
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Mar 2023 at 05:26 UTC
Updated:
6 Dec 2023 at 02:19 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
ranjit1032002Created a patch for Drupal 10 compatibility, please review.
Thank You.
Comment #4
hardikpandya commentedWith Drupal 10 here and reduced support for Drupal 8 going forward, I feel the info file should reflect the same. I have updated the
core_version_requirementaccordingly. Attached is the interdiff.Comment #5
marcelovaniThanks for the patches, but we are blocked by #3272732 https://www.drupal.org/project/entity_embed/issues/3272732
Comment #7
marcelovaniComment #8
marcelovaniComment #9
marcelovaniComment #10
urvashi_vora commentedFor Drupal 10 and CKEditor5 compatibility:
We could try the following steps:
1. In the paragraphs_inline_entity_form.module file, add the following code:
2. In the composer.json file, add the following line to require the CKEditor5 package:
3. Run composer update to install the CKEditor5 package.
4. In the paragraphs_inline_entity_form.libraries.yml file, add something like:
Comment #11
urvashi_vora commentedComment #12
marcelovaniComment #13
shivam_tiwari commentedComment #15
shivam_tiwari commentedComment #16
marcelovani@shivam_tiwari Thanks for the work on this, it will require some testing to validate the approach. Meanwhile, can you please rebase and update the pull request?
Comment #17
shivam_tiwari commented@marcelovani I am not getting any changes here after rebase. Please check and let me know if anything.
Comment #18
marcelovaniIf you go to https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge... you will see the messages

Try to resolve on Gitlab itself by clicking the link on the message
Comment #21
darvanenNeither of these branches seem to contain a CKEditor 5 plugin. It install on Drupal 10 fine but CKEditor 5 compatibility is not ready yet.
Comment #22
nelo_drup commentedModule status for drupal 5
Comment #23
darvanenOk clearly I didn't know this system nearly well enough to be commenting in #21 about there being no plugin, that's further up the chain. I stand by my comment on the MR about the library definition though.
Comment #24
mortona2k commentedI didn't dig too deep, but I suspect the plugin will need a rewrite to be compatible with cke5.
Comment #25
darvanenThis module does work with a cocktail of patches on its dependencies, but if you're hoping to insert entities in-text like our use-case you'll be blocked by the new `entity_embed` widget which only injects entities at the block level, between
elements, not inside them.
That said, as far this module is concerned, I'd say it passes manual testing (if you remove the 'ckeditor5' library definition from this module's library schema)
Comment #26
nelo_drup commentedLet's hope this module is updated because for now I can only use the following module https://www.drupal.org/project/paragraphs_entity_embed although it has its limitations
Comment #27
damienmckennaWould it be worth splitting the ckeditor 5 integration from the Drupal 10 compatibility? If nothing else it'd make the pieces easier to test on their own.
Comment #28
damienmckennaAlso, if the changes for API compatibility mean that it's no longer compatible with DRupal 8, the info file should be updated to note this.
Comment #29
marcelovaniThis project is very simple, it basically provides a custom Entity embed form that works as a glue to embed Paragraphs.
It depends on https://www.drupal.org/project/entity_embed/issues/3272732, that depends on https://www.drupal.org/project/embed/issues/3309747
Looks like these two issues are getting close to be resolved, but we are not there yet.
Comment #30
marcelovaniLooks like the dependencies have been fixed, I will try to allocate some time to check this.
Comment #31
marcelovaniComment #32
marcelovaniAttaching WIP PR https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge...
Currently blocked by https://www.drupal.org/project/entity_embed/issues/3396133
Comment #33
lexfunk commentedThank you everyone who has been contributing to this and all the related issues.
Comment #39
marcelovaniThanks for checking @lexfunk and everyone that commented and tested this. I have just merged the PR https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge...
I will make a new release shortly.
Comment #40
marcelovaniComment #42
marcelovaniUploaded a video on Youtube
Drupal 10 Demo Video
Comment #44
marcelovaniReopened to merge parts of https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/merge...
Comment #47
marcelovaniI partially merged this PR in https://git.drupalcode.org/project/paragraphs_inline_entity_form/-/commi... as the PR was not mergeable.
Comment #48
marcelovani