- Issue created by @akalam
- @akalam opened merge request.
- Status changed to Needs review
over 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
over 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.
- First commit to issue fork.
- 🇧🇪Belgium ant1
I created a initial setup for the submodule.
There is still the issue of data loss regarding this setting with the upgrade.