- Issue created by @narendraR
- Status changed to Postponed
10 months ago 12:14pm 29 February 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
I'm not at all convinced that we need to validate that every module is listed here. I think that not being listed here means that the module's simple config is installed (as it is required) and nothing more is okay.
I do think that it is worth discussing changing the name from config to config_entities as that's what this is about.
- Status changed to Active
10 months ago 12:27pm 29 February 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
I'm not at all convinced that we need to validate that every module is listed here. I think that not being listed here means that the module's simple config is installed (as it is required) and nothing more is okay.
I do think that it is worth discussing changing the name from config to config_entities as that's what this is about.
- 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Note: the scope of this issue first came up in #3413824: Warn Recipe developers if a module being installed does not have a corresponding config import declaration → .
I do think that it is worth discussing changing the name from config to config_entities as that's what this is about.
+1! This would go a long way.
But … I'd still like to try to convince you that we should require
config.import
for every installed module:- installing a module in Drupal always installs default config — both simple config and config entities
- … except in Recipes, only simple config is installed by default
- unless you specify
import: { some_module: '*' }
. This is not obvious.
It's both for discoverability and explicitness/declarativeness that IMHO
config.import.<module name>
should be present for every installed module. Because then the validation can inform the Recipe author that they should say which (if any) config entities of the installed module should be installed. - 🇺🇸United States thejimbirch Cape Cod, Massachusetts
But … I'd still like to try to convince you that we should require config.import for every installed module
In the last 6 months I think we have educated and documented why recipes don't behave like a regular module install, for recipe author specificity and idempotency.
I do think that it is worth discussing changing the name from config to config_entities as that's what this is about.
Do we want to change
config
toconfig_entities
to add further clarity?