Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Pathauto 1.3 was recently released:
https://www.drupal.org/node/2582001
The biggest and most important change for us, is that this patch is finally merged:
https://www.drupal.org/node/936222
We need to make sure that updates work. I'm particularly concerned that the hook_update_N() function may have been renumbered since the last version of the patch that we used.
There were also lots of other changes in this release, so we need to make sure that they don't cause any problems for other functionality (like the panopoly_pages customizations).
Comment | File | Size | Author |
---|---|---|---|
#10 | panopoly_pages-pathauto-2597273-10.patch | 934 bytes | dsnopek |
#9 | panopoly_core-pathauto-2597273-9.patch | 1.29 KB | dsnopek |
Comments
Comment #2
dsnopekComment #4
dsnopekWorked in my manual testing. I'm so excited about this change, that I've just committed it. ;-) And we'll see what Travis-CI says:
https://travis-ci.org/panopoly/panopoly/builds/88186696
Comment #5
dsnopekThis causes some Features to be overridden:
https://travis-ci.org/panopoly/panopoly/jobs/88186697#L629
This is breaking the tests. Reverting for now...
Comment #8
dsnopekHere's a patch. Need to try the tests on Travis again!
EDIT: Here's the Travis build: https://travis-ci.org/panopoly/panopoly/builds/166511599
Comment #9
dsnopekAssuming the same things that broke last time we ran the tests, then these patches might fix it
EDIT: It was the same failures! Here's a build with the new patches: https://travis-ci.org/panopoly/panopoly/builds/166514876
Comment #10
dsnopekSo, it's complaining that panopoly_pages should have:
... but that doesn't actually make sense, because panopoly_core is what is declaring that Taxonomy vocabulary - I'm not sure what panopoly_pages has to do with it.
But here's a patch to exclude that too, just so we can move this forward.
EDIT: Here's the new build: https://travis-ci.org/panopoly/panopoly/builds/166521744
Comment #12
dsnopekLast test build passed, so... committed! Finally!