- Issue created by @akalam
- @akalam opened merge request.
- Status changed to Needs review
about 1 year ago 8:09am 18 May 2023 - πͺπΈSpain akalam
Added a MR with the first approach: Add dependency on pathauto.
- π§πͺBelgium johanvdr
Based on https://www.drupal.org/project/node_keep/issues/3361172#comment-15056012 π Node keep can be wrongly installed without creating the alias_keeper field Needs work from @akalam I created a patch following the approach: Add dependency on pathauto.
- π§πͺBelgium johanvdr
Patch partially did not apply. Something with the last line and *.info.yml adding extra lines when performing composer install.
- Status changed to Needs work
about 1 year ago 3:59pm 23 June 2023 - π©πͺGermany Anybody Porta Westfalica
Pathauto should stay an optional dependency. Otherwise, that part of the functionality should be extracted into a submodule. The module shouldn't require to install pathauto.
- π§πͺBelgium antoineh Mechelen, π§πͺ
I created a initial setup for the submodule.
There is still the issue of data loss regarding this setting with the upgrade.