Launchpad itself

Merge lp://staging/~bryce/launchpad/lp-551848-270903-url-fixes into lp://staging/launchpad

  1. lp-551848-270903-url-fixes
  2. Merge into devel
Proposed by Bryce Harrington
Status: Merged
Approved by: Māris Fogels
Approved revision: no longer in the source branch.
Merged at revision: 11048
Proposed branch: lp://staging/~bryce/launchpad/lp-551848-270903-url-fixes
Merge into: lp://staging/launchpad
Diff against target: 22 lines (+2/-2)
2 files modified
lib/canonical/launchpad/mail/errortemplates/no-default-affects.txt (+1/-1)
lib/lp/bugs/templates/malone-portlet-bugmail-filtering-faq.pt (+1/-1)
To merge this branch: bzr merge lp://staging/~bryce/launchpad/lp-551848-270903-url-fixes
Reviewer Review Type Date Requested Status
Māris Fogels (community) Approve
Review via email: mp+27592@code.staging.launchpad.net

Commit message

Fixes two URLs that are exposed in the launchpad ui.

Description of the change

This fixes a couple URLs that are exposed in the launchpad ui.

To post a comment you must log in.
Revision history for this message
Māris Fogels (mars) wrote :

Hi Bryce,

The code change looks good, but has it been tested? There may be a test or two that depend on the mail footer text.

r=mars, conditional on the branch passing the test suite.

Maris

review: Approve
Revision history for this message
Bryce Harrington (bryce) wrote :

Yes, it passed the ec2 test without issue; I guess there were no dependencies by the tests on this text.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.