Code review comment for lp:~bac/launchpad/bug-435599

Revision history for this message
Brad Crittenden (bac) wrote :

= Summary =

During the 3.0 release a link to the blog was broken because it used the id-based URL
rather than the longer human-readable, permalink URL. This is described in bug 4335599.

== Proposed fix ==

The change here is trivial to use the longer URLs. Hopefully getting the id-based
ones out of the template will discourage their future use.

== Pre-implementation notes ==

Chat with Curtis.

== Implementation details ==

Straightforward.

== Tests ==

No

== Demo and Q/A ==

Look at the front page and ensure the new URLs are used.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/canonical/launchpad/templates/root-index.pt

« Back to merge proposal