Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
As a distribution we want to specify a custom url for the first page after installation.
Proposed resolution
Add a key to the profile.info.yml -> distribution[install][finish_url]
Comment | File | Size | Author |
---|---|---|---|
#18 | interdiff-2776605-15-18.txt | 688 bytes | chr.fritsch |
#18 | 2776605-18.patch | 2.63 KB | chr.fritsch |
#15 | interdiff-2776605-13-15.txt | 1.36 KB | chr.fritsch |
#15 | 2776605-15.patch | 1.96 KB | chr.fritsch |
#13 | interdiff-2776605-2-13.txt | 778 bytes | chr.fritsch |
Comments
Comment #2
chr.fritschComment #3
chr.fritschComment #5
dawehnerSeems like a total valid feature request. Is there an test distribution we could leverage to test that kind of behaviour in core?
Comment #6
dawehnerComment #7
phenaproxima+1 for this feature! And +1 on the need for tests...although I have no idea how to approach that.
One question: why would this functionality be specific to distributions? Couldn't we apply it to all profiles?
Comment #8
chr.fritschOne question about the tests. Could that be tested in DistributionProfileTest?
Comment #9
dawehnerWell some of the existing install profile / distribution test :)
Comment #13
chr.fritschAdded a small test
Comment #14
phenaproximaI like it, but I wonder if we could make the test a little more solid. For example, let's have the test distro create a node or some other URL which definitely does not exist in Drupal core, and have it redirect to that. That would prove with 100% certainty that the redirection is working as intended.
Comment #15
chr.fritschWhat do you think about this?
Comment #16
phenaproximaI think that’s pretty much RTBC, buddy. Will revisit when the tests pass.
Comment #17
phenaproximaYup. Me likely. Let's land it.
Comment #18
chr.fritschI drafted a change record and just updated some documentation.
Comment #19
alexpottCrediting @dawehner and @phenaproxima for suggesting tests and improvements to them.
Comment #20
alexpottCommitted 81ef7a6 and pushed to 8.6.x. Thanks!