- Issue created by @mogtofu33
- Merge request !68Issue #3540207 by mogtofu33, pdureau, grimreaper: Check LayoutBuilder core patch for Entity view → (Merged) created by Grimreaper
- 🇫🇷France Grimreaper France 🇫🇷
Please test if newly referenced patch applies.
Updated doc to recommend composer patches.
- 🇫🇷France Grimreaper France 🇫🇷
Documentation mentioned only composer patches 1: https://www.drupal.org/docs/develop/using-composer/manage-dependencies#p... →
-
mogtofu33 →
committed 0be59308 on 1.0.x authored by
grimreaper →
Issue #3540207 by mogtofu33, pdureau, grimreaper: Check LayoutBuilder...
-
mogtofu33 →
committed 0be59308 on 1.0.x authored by
grimreaper →
- 🇫🇷France mogtofu33
Let force patches 1, I kept the merge in doc as it's still relevant for third party like ui_patterns.
Thanks for the work! - 🇫🇷France Grimreaper France 🇫🇷
With
"enable-patching": true,
there is no need to duplicate the patches in the project composer.json - 🇫🇷France mogtofu33
You mean in the composer.json of the module?
For me it's not working, the patch need to be in the main, it's not merge even with"enable-patching": true
. - 🇫🇷France Grimreaper France 🇫🇷
If patch is in the composer.json of the module, with enable-patching: true in the composer.json of a project, no need to duplicate patches in the composer.json of the project.
Strange, I use this approach on a client project, on Sobki and in ui_styles, and it works.
Let's keep it that way and no more touch stuff.