Don't record changed file captions for diff file attachments.

Review Request #13673 — Created April 10, 2024 and submitted — Latest diff uploaded

Information

Review Board
release-7.x

Reviewers

Publishing updates to diffs that included file attachments was showing a
"File Captions" field in the change description that included captions
for every file in every commit in the new diff. These files are already
listed in the "Diff" field, and they don't have editable captions at all.

This change filters them out when we're deciding what to record in the
change description.

Posted an update to a change that included binary files. Saw that there
was no longer a bunch of "File Caption" changes in the change description.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Don't record changed file captions for diff file attachments.
Publishing updates to diffs that included file attachments was showing a "File Captions" field in the change description that included captions for every file in every commit in the new diff. These files are already listed in the "Diff" field, and they don't have editable captions at all. This change filters them out when we're deciding what to record in the change description. Testing Done: Posted an update to a change that included binary files. Saw that there was no longer a bunch of "File Caption" changes in the change description.
120ee59d75d812c727da2c90845e3ac3afd2af44 David Trowbridge
reviewboard/reviews/models/review_request_draft.py
Loading...