Merge lp:~mthaddon/juju-website/version-2-fix-https-favicon into lp:~canonical-website-editors/juju-website/version-2
Proposed by
Tom Haddon
Status: | Merged |
---|---|
Merged at revision: | 28 |
Proposed branch: | lp:~mthaddon/juju-website/version-2-fix-https-favicon |
Merge into: | lp:~canonical-website-editors/juju-website/version-2 |
Diff against target: |
19 lines (+2/-2) 1 file modified
header.php (+2/-2) |
To merge this branch: | bzr merge lp:~mthaddon/juju-website/version-2-fix-https-favicon |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Anthony Dillon | Approve | ||
Review via email:
|
Description of the change
Per https:/
This fixes juju.ubuntu.com so that the favicon comes from https if the rest of the site is being served on https to avoid browser warnings. I've cowboyed this place in change on juju.ubuntu.com to confirm, so when you're ready to rollout, you'll want to note that in the deployment RT.
To post a comment you must log in.
LGTM. Thank you very much.