- Issue created by @deviantintegral
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
over 1 year ago Not currently mergeable. - @deviantintegral opened merge request.
- Status changed to Needs review
over 1 year ago 2:02pm 13 September 2023 - last update
over 1 year ago 46 pass, 2 fail - 🇨🇦Canada deviantintegral
Gitlab seems to be having trouble this morning. Here's a patch just in case my push doesn't eventually make it to the merge request.
The last submitted patch, 4: 3387049-increase-maximum-default.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- last update
over 1 year ago 48 pass - 🇨🇦Canada deviantintegral
Fixes unrelated but now required code style issues.
- Status changed to Needs work
over 1 year ago 2:14pm 8 October 2023 - 🇨🇭Switzerland berdir Switzerland
The second patch seems to not include the actual change?
Nice work on checking related issues and history on this. Fine with changing the default, but IMHO the update function is not necessary. If someone cares, they'll also see this in the release notes, and that increases the risk of people using patches with update functions for minimal gain.
I'm in the process of switching to gitlabCI, so merge requests are preferred over patches, would be great to have that updated.
- 🇨🇦Canada deviantintegral
Yeah, agreed on MRs, I think with the recent hosting changes my prior issues should be solved. I'll file one without the update hook.
- last update
over 1 year ago 48 pass - @deviantintegral opened merge request.
- last update
over 1 year ago 48 pass - Status changed to Needs review
over 1 year ago 3:03pm 19 October 2023 - 🇮🇳India emilymathew
I have applied the merge request !58 in #9 in pathauto version 8.x-1.12 in my Drupal 10.2.2 installation. The changes worked for me. The default Maximum alias length now updated to 224 from 100.
Attaching the screenshots before and after. - Status changed to RTBC
about 1 year ago 11:46am 27 January 2024 - Issue was unassigned.
We're using this patch on a customer website from a long time and it's working fine for us without any problem.
RTBC + 1