Problem/Motivation

Pathauto is a widely used module and we should ensure we integrate properly with it.

Proposed resolution

Add tests including pathauto.

Remaining tasks

Tests.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#7 2804661-7.patch12.8 KBpenyaskito
#2 2804661-2.patch13.05 KBpenyaskito
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Active » Needs review
FileSize
2804661-2.patch13.05 KB

Attached patch with tests.

Status: Needs review » Needs work

The last submitted patch, 2: 2804661-2.patch, failed testing.

The last submitted patch, 2: 2804661-2.patch, failed testing.

The last submitted patch, 2: 2804661-2.patch, failed testing.

  • penyaskito committed 50d34f6 on 8.x-1.x 
    Issue #2804661 by penyaskito: Add Pathauto to test_dependencies
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Needs work » Needs review
FileSize
2804661-7.patch12.8 KB

Committed the change to test_dependencies so testbot can take it. Adding the patch with the test only.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Needs review » Reviewed & tested by the community

Passed as expected.

  • penyaskito committed cc40a6d on 8.x-1.x 
    Issue #2804661 by penyaskito: Ensure lingotek works with Pathauto
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Reviewed & tested by the community » Fixed

Committed cc40a6d and pushed to 8.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.