launchpad-reviewers team mailing list archive

[Toto Bille <thomas.bille@xxxxxxxxxxxxx>]

Toto Bille has proposed merging ~tbmb/launchpad:update-links into launchpad:master.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~tbmb/launchpad/+git/launchpad/+merge/386329

Update links to new careers page.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~tbmb/launchpad:update-links into launchpad:master.

diff --git a/lib/lp/app/browser/doc/base-layout.txt b/lib/lp/app/browser/doc/base-layout.txt
index ddf10c1..f9eb8b7 100644
--- a/lib/lp/app/browser/doc/base-layout.txt
+++ b/lib/lp/app/browser/doc/base-layout.txt
@@ -150,7 +150,7 @@ Page Footers
     Data privacy https://www.ubuntu.com/legal/dataprivacy
     Contact Launchpad Support /feedback
     Blog http://blog.launchpad.net/
-    Careers http://www.canonical.com/about-canonical/careers
+    Careers https://canonical.com/careers
     System status https://twitter.com/launchpadstatus
     Get the code! https://dev.launchpad.net/
 
diff --git a/lib/lp/app/templates/base-layout-macros.pt b/lib/lp/app/templates/base-layout-macros.pt
index 938d47d..175551d 100644
--- a/lib/lp/app/templates/base-layout-macros.pt
+++ b/lib/lp/app/templates/base-layout-macros.pt
@@ -301,7 +301,7 @@
 	>Blog</a>
       <tal:careers_link condition="not: features/baselayout.careers_link.disabled">
 	&nbsp;&bull;&nbsp;
-	<a href="http://www.canonical.com/about-canonical/careers";
+	<a href="https://canonical.com/careers";
 		>Careers</a>
       </tal:careers_link>
       &nbsp;&bull;&nbsp;