Merge lp://staging/~mnordhoff/loggerhead/relative-links into lp://staging/loggerhead
Proposed by
Matt Nordhoff
Status: | Merged |
---|---|
Approved by: | Martin Albisetti |
Approved revision: | 419 |
Merged at revision: | 416 |
Proposed branch: | lp://staging/~mnordhoff/loggerhead/relative-links |
Merge into: | lp://staging/loggerhead |
Diff against target: |
101 lines (+19/-14) 5 files modified
loggerhead/apps/branch.py (+10/-5) loggerhead/apps/config.py (+2/-2) loggerhead/apps/transport.py (+2/-2) loggerhead/controllers/download_ui.py (+2/-2) loggerhead/templates/atom.pt (+3/-3) |
To merge this branch: | bzr merge lp://staging/~mnordhoff/loggerhead/relative-links |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Albisetti | Approve | ||
Review via email: mp+24767@code.staging.launchpad.net |
Commit message
Generate relative links where possible (Michael Hudson, Matt Nordhoff)
Description of the change
Quoting https:/
> This branch generates relative links apart from in redirects.
*This* branch fixes the issues I noticed when reviewing that branch many moons ago, namely that absolute links are necessary in some parts of the Atom feed. The original branch fixed one HTTP redirect to be absolute; this branch fixes the rest too.
There may be other issues. I dunno.
To post a comment you must log in.
Updating diff...
An updated diff will be available in a few minutes. Reload to see the changes.
Looks good enough to eat, to me.