- Issue created by @Rajab Natshah
- Issue was unassigned.
- π―π΄Jordan Rajab Natshah Jordan
Not sure if this issue should be moved to the Component Libraries: Blocks project.
- @rajab-natshah opened merge request.
- Status changed to Needs review
over 1 year ago 11:54am 18 June 2023 - e0ipso Can Picafort
I am surprised this is the only issue you encountered so far. CL Block is still not compatible with the 2.x series and with SDC in core.
Can we turn this issue into a more generic issue for SDC compatibility?
- π―π΄Jordan Rajab Natshah Jordan
Thanks for passing by this issue.
Yes, for sure if you like that.
I had one issue before this one
π Fix Block Settings TypeError: array_filter() when pressing Save configuration Needs reviewToday I faced the following issue too.
π Fix TypeError when editing Text formats and editors for the Cl Component Embed filter having Argument #1 ($input) must be of type array, null given ActiveIt feels that a one refactor issue could manage refactoring the code to support SDC from Drupal
~10.1.0
and CL family of modules to be inline with the2.0.x
branches.More testing rounds and experimentation with all dynamic config and behaviors.
- π¨π΄Colombia jidrone
I agree with @e0ipso, I was trying to make it work with the latest version of Drupal core and all other modules and I have found more things to fix, so instead of having a lot of small issues that don't work independently I would rename this for all the compatibility issues.
Once the patch get ready maybe we can create a 2.x version.
- Status changed to Needs work
over 1 year ago 12:17am 17 August 2023 - @jidrone opened merge request.
- Status changed to Needs review
over 1 year ago 8:09pm 17 August 2023 - π¨π΄Colombia jidrone
I started a new issue fork and MR with multiple fixes:
- Changed references to CL components to SDC
- Removed variants functionality
- Removed CL Components dependency
- Multiple fixes to get the component schema
There are some things I didn't test yet, like the CK editor embed, I will move it to needs review for now.
- e0ipso Can Picafort
@jidrone it really shows that you have been getting familiar with the codebase! This MR is absolutely fantastic!
I left a bunch of comments in the MR.
Some overarching questions & requests:
- Drupal 10 will use CKEditor5, but I didn't see any changes in this department. Perhaps we can add a dependency to https://www.drupal.org/project/ckeditor β instead?
- We now have
sdc_tags
as part ofcl_editorial
to deal with the configuration form on what components are eligible for each feature. Can we also migrate to that as well, or create a ticket to do so? There is an example on how to do it insdc_display
- Can you post screenshots of the module working after the patch? This will help identify the features you have tested.
@jidrone I think this is great for Layout Builder workflows and sites sticking behind with CKEditor4, but in the future I'd like to break this module in two:
sdc_block
andsdc_ckeditor
. What do you think about this? - Status changed to Needs work
over 1 year ago 5:28pm 7 September 2023 - π¨π΄Colombia jidrone
Yes, I think the CKEditor integration should be in a separate module.
- πΊπΈUnited States dave reid Nebraska USA
Dave Reid β made their first commit to this issueβs fork.