- Issue created by @jayhuskins
- Merge request !8Update sdc_display.info.yml to include SDC as a dependency β (Open) created by jayhuskins
- Status changed to RTBC
5 months ago 10:25am 25 January 2024 - πͺπΈSpain pcambra Spain, πͺπΊ
Yup.
Otherwise you get
" The service "Drupal\cl_editorial\NoThemeComponentManager" has a dependency on a non-existent service "plugin.manager.sdc"."
- Status changed to Needs review
5 months ago 5:15pm 25 January 2024 - e0ipso Can Picafort
I am hesitant to merge this, because this module will soon be removed from core (and merged into the Core namespace instead). This will cause this module to download the contrib module instead.
How do you think we should handle that?
- πͺπΈSpain pcambra Spain, πͺπΊ
I was not aware of that, we either merge this and remove it and release a minor whenever that happens or we ignore/postpone it
- Status changed to RTBC
5 months ago 6:47pm 29 January 2024 - πΊπΈUnited States jayhuskins
SDC has already been merged into core as an experimental module. This MR refers to the core version "drupal:sdc". The contrib version would be referred to as "sdc:sdc".
- Status changed to Needs review
5 months ago 9:22am 30 January 2024 - πͺπΈSpain pcambra Spain, πͺπΊ
@jayhuskins this is not what @e0ipso is referring to, please see π± [META] Move code from the experimental SDC module to core Active
- πΊπΈUnited States brianperry
On the current 10.4.0-dev if I install Drupal, and then install this module I get the error that @pcambra mentioned. I'm only able to use this module if I enable the experimental SDC module.
So I don't think we'd want to add the experimental SDC module as a dependency - my understanding is that going forward components / SDCs are just a fundamental part of Drupal that don't need to be enabled. Instead, we need to figure out what this sdc_display is relying on from the experimental module and fix that.