Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lolandese’s picture

Status: Active » Needs review
FileSize
3.3 KB

Patch provided that can be deployed by going to:
http://simplytest.me/project/permalink_block/7.x-1.x?add[]=popup&patch[]=https://drupal.org/files/demo-2043135-1.patch

This patch creates a sample page and some settings in a temporary sandbox on http://simplytest.me/ and is not intended to be committed to the module itself.

lolandese’s picture

Install breaks on line 62 permalink_block.install:

// Go to the sample page.
  drupal_goto('node/1');

After site slogan, before installation message.

Remove and retry.

lolandese’s picture

lolandese’s picture

Status: Needs work » Fixed
FileSize
4.35 KB

It works. We have to renounce to the final redirect to a node page. That would have been better for demo purpose, but we end up on the frontpage. Might be interesting to look into this.

We set it fixed for now, as it is working. Some slight changes, so a new and probably last patch for now. That'll be the one we should use for the demo link on the project page:
Try out a demonstration

lolandese’s picture

FileSize
4.27 KB

Some other small aesthetic changes.

Try out a demonstration

Status: Fixed » Closed (fixed)

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

lolandese’s picture

Issue summary: View changes
Status: Closed (fixed) » Active

Reroll needed for the latest dev.

lolandese’s picture

Status: Active » Needs review
FileSize
4.75 KB

Reroll against latest dev.

lolandese’s picture

Status: Needs review » Fixed
lolandese’s picture

Status: Fixed » Needs review
FileSize
4.48 KB

A reroll against latest dev.

lolandese’s picture

Status: Fixed » Closed (fixed)

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