Merge lp://staging/~stevanr/linaro-license-protection/bug1085007 into lp://staging/~linaro-automation/linaro-license-protection/trunk

Proposed by Stevan Radaković
Status: Merged
Merged at revision: 164
Proposed branch: lp://staging/~stevanr/linaro-license-protection/bug1085007
Merge into: lp://staging/~linaro-automation/linaro-license-protection/trunk
Diff against target: 59 lines (+18/-20)
2 files modified
templates/header.html (+9/-10)
templates_releases/header.html (+9/-10)
To merge this branch: bzr merge lp://staging/~stevanr/linaro-license-protection/bug1085007
Reviewer Review Type Date Requested Status
Paul Sokolovsky Approve
Linaro Infrastructure Pending
Review via email: mp+149764@code.staging.launchpad.net

Description of the change

Removed stale css file.
Introduced protocol-relative links for www.linaro.org.
Changed js/css scripts to hostless absolute paths.

To post a comment you must log in.
Revision history for this message
Paul Sokolovsky (pfalcon) wrote :

Looks good. I didn't known about protocol-relative URLs, found confirmation here: http://stackoverflow.com/questions/4978235/absolute-urls-omitting-the-protocol-scheme-in-order-to-preserve-the-one-of-the . That probably won't solve all issues we have (we have wrong SSL certificate for www.linaro.org), but at least it should resolve biggest probs.

review: Approve

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.

Subscribers

People subscribed via source and target branches