Make a bunch of improvements to the FileAttachment admin list.

Review Request #13576 — Created Feb. 27, 2024 and submitted — Latest diff uploaded

Information

Review Board
release-7.x

Reviewers

The way attachments were listed in the admin site was very difficult to
use. The file field was linking to the download URL, and the only
other link to the change page was the caption, which could be empty.

This change improves the admin list so that the filename rather than the
path is listed (potentially cropped to preserve the page with), but that
always links to the change page. The "Review Request ID" column now
properly shows the linked review request even when the attachment is
part of a draft or in the inactive list. A new column has been added
with the download link.

Used the admin list and was actually able to get to the change page for
many of my attachments.

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.

orig
1
2

Commits

First Last Summary ID Author
Make a bunch of improvements to the FileAttachment admin list.
The way attachments were listed in the admin site was very difficult to use. The `file` field was linking to the download URL, and the only other link to the change page was the caption, which could be empty. This change improves the admin list so that the filename rather than the path is listed (potentially cropped to preserve the page with), but that always links to the change page. The "Review Request ID" column now properly shows the linked review request even when the attachment is part of a draft or in the inactive list. A new column has been added with the download link. Testing Done: Used the admin list and was actually able to get to the change page for many of my attachments.
120c4977ea649e4a9f1c2ac0f9fae108fd623252 David Trowbridge
reviewboard/attachments/admin.py
Loading...