Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#20678 closed Bug (fixed)

Broken link for downloading docs as HTML

Reported by: jennyd Owned by: jezdez
Component: * Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


The link to download the full docs as HTML at the bottom of the sidebar on many pages on is given as e.g. but should be - this applies to the links for all versions of Django.

Change History (7)

comment:1 Changed 3 years ago by bmispelon

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 3 years ago by claudep

This is an artifact of the static template tag, introduced in

comment:3 Changed 3 years ago by bmispelon

This issue was introduced with commit that switched the site to using CachedStaticFilesStorage.

Unfortunately, the code that generates thoses zip files (the update_docs command) is not using the staticfiles machinery, which broke the download links.

I see two solutions:

1) Revert the download link to use {{ STATIC_ROOT }} instead of {% static %}.
2) Change the update_docs command to make use of the staticfiles infrastructure (or at least generate a compatible filename).

I think 2) might be the better option, but after looking into it a bit, I found it tricky to implement.

comment:4 Changed 3 years ago by aaugustin

It could be more appropriate to treat the docs zip archives as media files.

There're a media attached to a DocumentRelease object.

comment:5 Changed 3 years ago by jezdez

  • Owner changed from nobody to jezdez
  • Status changed from new to assigned

comment:6 Changed 3 years ago by Jannis Leidel <jannis@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:7 Changed 3 years ago by jezdez

FTR I agree with aaugustin, the download is a "media" IMO.

Note: See TracTickets for help on using tickets.
Back to Top