- Issue created by @Anybody
- π¬π§United Kingdom aaron.ferris
aaron.ferris β made their first commit to this issueβs fork.
- π¬π§United Kingdom aaron.ferris
I was going to start this, but do we think it'd be worth waiting for some of the bulkier changes to be in before doing this, to avoid conflicts etc?
- Assigned to aaron.ferris
- Issue was unassigned.
- Status changed to Postponed
8 months ago 6:13pm 7 April 2024 - π©πͺGermany Anybody Porta Westfalica
Thanks @aaron.ferris yes let's wait for that! It's also optional with no "real" benefit.
- π«π·France erwangel
Although that's confusing for people wanting to install the module. You require it with composer as "front" but you have it to enable it with drush as "front_page". confusing and not very intuitive when drush tells you "Unable to install modules front due to missing modules front." I had to look in the modules's directory to realize that its "real" name was "front_page" and not "front". I think a quick fix will be useful so more people could install it and you could gain more feedback.
- π¦πΊAustralia elc
Had a thought about this as it will be a rather large breaking change.
- Copy the entire module into a sub-dir called "modules"
- Rename all of the base dir module to the correct file and function names (and classes so as not to dup anything)
- Add an update to the sub-module to install "front"
- Add an enable to base dir module to uninstall "front_page" if it's installed
- Add a hook_requirements() to sub-dir module to ensure it can't be installed to start with
Haven't figured out the intricate details, or if this would work at all but it seems like it would. Use the lifecycle on the sub-dir module.
Agree with holding off until majority of changes are in so as not to increase the amount of work needed to do this.