- Issue created by @ciprian.stavovei
- Merge request !541Issue #3523920: Subsequent recipes installation breaks because of gin config... β (Merged) created by ciprian.stavovei
- πΊπΈUnited States phenaproxima Massachusetts
I agree; there's no particular reason for this recipe to compare config strictly. RTBC if tests pass.
- πΊπΈUnited States phenaproxima Massachusetts
On second thought...although I approve of the fix, I am kind of wondering how this is happening. The drupal_cms_admin_ui recipe, like all our recipes, has a test which proves it can be applied twice. So something else must be modifying the block and making it deviate from our shipped config.
Again - I think that setting
strict: false
is completely fine, and the MR is RTBC. But I want to more thoroughly understand the problem before I merge this, just in case it points to a larger bug that needs a more comprehensive fix.Can you provide steps to reproduce, starting from a bare Drupal CMS site -- or whatever it is you started with?
- π¦πΉAustria arthur_lorenz Vienna
Thanks, we are currently working on creating recipes β based on Drupal CMS for our Lupus Decoupled β stack. To provide a streamlined editing experience we base all our recipes on the
drupal_cms_starter
recipe. That's how we noticed that issue.To reproduce it using only
drupal_cms
recipes:* Fresh Drupal install
* Applydrupal_cms_starter
recipe
* Applydrupal_cms_news
recipe
* Applydrupal_cms_starter
recipe againIt fails because the
gin_messages
-block's weight got altered. - πΊπΈUnited States phenaproxima Massachusetts
we base all our recipes on the
drupal_cms_starter
recipeSo this is the one recipe in our stack which is not meant to be applied more than once. I would suggest not using this recipe, and instead compose your recipe using the smaller parts of Drupal CMS (such as
drupal_cms_admin_ui
, or any of the other recipes we have, except fordrupal_cms_starter
). - πΊπΈUnited States phenaproxima Massachusetts
But, having said that, I think I'm still okay merging this patch. There's no harm in making
drupal_cms_admin_ui
non-strict. -
phenaproxima β
committed eca83bd5 on 1.x authored by
ciprian.stavovei β
Issue #3523920 by ciprian.stavovei, arthur_lorenz: drupal_cms_admin_ui...
-
phenaproxima β
committed eca83bd5 on 1.x authored by
ciprian.stavovei β
-
phenaproxima β
committed ccc43ed8 on 1.1.x authored by
ciprian.stavovei β
Issue #3523920 by ciprian.stavovei, arthur_lorenz: drupal_cms_admin_ui...
-
phenaproxima β
committed ccc43ed8 on 1.1.x authored by
ciprian.stavovei β
- πΊπΈUnited States phenaproxima Massachusetts
Merged into 1.x and cherry-picked to 1.1.x. Thanks!