- Issue created by @joachim namyslo
- ๐ฎ๐ณIndia kkalashnikov Ghaziabad, India
kunalgautam โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia kkalashnikov Ghaziabad, India
Hi @joachim namyslo It would be better to remove the ai_image_alt_text configuration from the AI recipe.
- Merge request !546Issue #3525114 by kunalgautam: Remove ai_image_alt_text config from ai recipe โ (Merged) created by kkalashnikov
- ๐ฉ๐ชGermany joachim namyslo Kulmbach ๐ฉ๐ช ๐ช๐บ
I am not sure if it'll fix it. That's why I opened the bug report. If it fix that I could use some help to get recipes back.
- First commit to issue fork.
- ๐บ๐ธUnited States thejimbirch Cape Cod, Massachusetts
The configuration 'image.style.ai_image_alt_text' exists already and does not match the recipe's configuration
This error is happening because you already had an image style named
ai_image_alt_text
, but it was changed and different from the one the module provides.I reviewed the recipe and the module's config, and they doesn't seem to be any reason to treat the config strict, so I made it lenient.
To validate, apply the recipe, change a setting in the image style, and then re-apply the recipe.
- ๐บ๐ธUnited States phenaproxima Massachusetts
I think this is an appropriate fix but I'd like to try reproducing this locally just to see what exactly is changing. It sounds like it can be reproduced on a fresh install of Drupal CMS.
- ๐บ๐ธUnited States phenaproxima Massachusetts
I can reproduce this, but only if I go actively make a change to the
AI Image Alt Text
image style. If I don't do that, there is no problem.With the problem there, if I add
strict: false
to the AI recipe, the list of projects comes back.Even if I don't know how exactly the image style is being changed in @joachim namyslo's set-up, it doesn't really matter. There's no reason for the AI recipe to be using strict config comparison.
-
phenaproxima โ
committed ae0acde8 on 1.x authored by
kunalgautam โ
Issue #3525114 by thejimbirch, joachim namyslo: The AI recipe should do...
-
phenaproxima โ
committed ae0acde8 on 1.x authored by
kunalgautam โ
-
phenaproxima โ
committed b470c073 on 1.1.x authored by
kunalgautam โ
Issue #3525114 by thejimbirch, joachim namyslo: The AI recipe should do...
-
phenaproxima โ
committed b470c073 on 1.1.x authored by
kunalgautam โ
- ๐บ๐ธUnited States phenaproxima Massachusetts
Merged into 1.x and cherry-picked to 1.1.x. Thanks!
- ๐ฉ๐ชGermany joachim namyslo Kulmbach ๐ฉ๐ช ๐ช๐บ
Thank you guys I do not know how this happened but I'll take a look at the patch and try to apply the change.
Automatically closed - issue fixed for 2 weeks with no activity.