- π¬π§United Kingdom catch
The idea was to allow recipes to install node module with a text_long body field, but recipes can already do that since they skip installing module config. So yeah this doesn't do anything, we need π Remove node_add_body_field from NodeTypeForm Active instead. Closing as duplicate of the other two isseus.
- π΅π±Poland dmitry.korhov Poland, Warsaw
If that's for Drupal CMS and not for every installation, why not to use separate `node/config/install` sub-folder that will install the relevant config?
- π¨πSwitzerland berdir Switzerland
I don't think that changes much in practice so far, if it doesn't have dependencies that might not be fullfilled, still going to be always installed?
If it's optional, we need to change node_add_body_field() to return early if the field doesn't exist and document that it might return null, which is a bit of an API change?
- πΊπΈUnited States phenaproxima Massachusetts
Okay - I merged !213, and will tag beta1 from that, but that's not the end of this issue. Let's deal with the rest of the dependencies/patches in subsequent MRs in this issue.
-
phenaproxima β
committed 02dbeb2b on 0.x authored by
jurgenhaas β
Issue #3488490 by phenaproxima, jurgenhaas: Switch to tagged releases of...
-
phenaproxima β
committed 02dbeb2b on 0.x authored by
jurgenhaas β
- π¬π§United Kingdom catch
Already deleteable via the API, just not via the UI.
- @catch opened merge request.
- Issue created by @catch
- π©πͺGermany jurgenhaas Gottmadingen
@phenaproxima MR!213 contains the switch to tagged releases for ECA, bpmn_io and Klaro.
- @jurgenhaas opened merge request.
- First commit to issue fork.
- Issue created by @phenaproxima