Problem/Motivation

The test fails currently because setting the custom frontpage is broken and the test can't assert the text on the front page as intended.

Proposed resolution

Fix setting the frontpage to /monitoring-demo, fix the test if necessary.

Remaining tasks

Do it, provide patch, review and commit it.

User interface changes

None

API changes

None

Data model changes

None

CommentFileSizeAuthor
#1 demo_module_frontpage-2515512-1.patch613 bytesLKS90
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LKS90’s picture

LKS90 CreditAttribution: LKS90 commented
Status: Active » Needs review
FileSize
demo_module_frontpage-2515512-1.patch613 bytes

Here is the simple patch, a missing slash in front of the path.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs review » Fixed

Committed, thx.

  • miro_dietiker committed bb75751 on 8.x-1.x authored by LKS90 
    Issue #2515512 by LKS90: Demo module frontpage is not set on install

Status: Fixed » Needs work

The last submitted patch, 1: demo_module_frontpage-2515512-1.patch, failed testing.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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