- Issue created by @arkener
- ๐ฌ๐งUnited Kingdom MrDaleSmith
The maintainers are reluctant to enforce module-level dependencies that aren't required by every plugin used by their code, so I think we will need to implement a system like https://www.drupal.org/project/ai_agents/issues/3515453 ๐ Agents have silent dependencies Active for the AiCKEditor plugins. This will take longer, but will allow dependencies to be controlled by the plugins that need them.
- Merge request !562Allow CKEditor plugins to declare dependencies on modules. #3519699 โ (Open) created by MrDaleSmith
- ๐ง๐ชBelgium leanderjcc Ghent
Reviewed and looks good!
After applying the patch, I no longer receive the error following the reproduce steps.
D11.1.6
- ๐ฉ๐ชGermany marcus_johansson
Thanks Paul, this looks good - but has a merge conflict, could you have a look and ping me and I will merge.
Feel free to merge with ๐ Don't show empty vocabularies [AI Content Suggestions] Active , so they don't cause merge conflicts as well, since both are "pre-approved".
- ๐ฎ๐ณIndia prashant.c Dharamshala
prashant.c โ made their first commit to this issueโs fork.
- ๐ฎ๐ณIndia prashant.c Dharamshala
@marcus
I have done the following:
- Rebased the branch, resolved conflicts
- Cherry picked commits from https://git.drupalcode.org/project/ai/-/merge_requests/574/commits, hope that is okay to prevent conflicts
- ๐ฉ๐ชGermany marcus_johansson
The code lloks good, however since this is in needs work, I will set this also here since its merged: https://git.drupalcode.org/project/ai/-/merge_requests/574/commits unless that was fixed as well @prashant.c?