- Issue created by @thejimbirch
- ๐บ๐ธUnited States phenaproxima Massachusetts
It is already the case that, in recipe.yml, you HAVE to say what to do with the config of each module your recipe installs. There is even validation code for it.
- ๐จ๐ฆCanada b_sharpe
I believe on points 2 and 3 it should note that this means you are overriding the default module installation and point out the possibility of unwanted affects
- ๐บ๐ธUnited States thejimbirch Cape Cod, Massachusetts
Should be added to:
https://git.drupalcode.org/issue/distributions_recipes-3381259/-/blob/33...We also decided that we could use NULL to bring in only the module's simple config. That should be tested and documented.
- ๐จ๐ฆCanada mandclu
Adding to the IS to mention that config actions are able to modify any module config imported. I still personally believe that this speaks to importing all module config as a best practice, but have this discussion document is tremendously helpful.
- ๐บ๐ธUnited States thejimbirch Cape Cod, Massachusetts
thejimbirch โ changed the visibility of the branch 3454444-document-the-reason to hidden.
- ๐บ๐ธUnited States thejimbirch Cape Cod, Massachusetts
thejimbirch โ changed the visibility of the branch 3454444-author-guide to hidden.
- Status changed to Needs review
6 months ago 3:24pm 14 June 2024 - Status changed to Fixed
5 months ago 10:23am 31 July 2024 - ๐ฎ๐ณIndia prashant.c Dharamshala
thejimbirch โ credited Prashant.c โ .
- ๐บ๐ธUnited States thejimbirch Cape Cod, Massachusetts
Thanks for the great reviews! Merged.
Automatically closed - issue fixed for 2 weeks with no activity.