[WIP] Add an is_draft property to file attachments.

Review Request #13138 — Created July 13, 2023 and discarded — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

This adds an is_draft property to file attachments, to easily check
when a file attachment is a draft or if it's published. This also
updates areas in the code where we check if a file attachment is a
draft. And unit tests for the file attachment resources have been
updated to check all of the fields of the file attachment.

  • Ran unit tests.
  • Used the property in upcoming changes.
  • Tested uploading user file attachments to comments, saw
    nothing out of the ordinary.

Changes between revision 2 and 3

orig
1
2
3
4
5
6
7
8

Commits

Summary ID Author
Add an is_draft property to file attachments.
867734ed832959a4d13425352fc5dda29dfca12a Michelle Aubin
Add an is_draft property to file attachments.
8bf7ce8d7fd6b9ad63838f29e3368f309460b7e8 Michelle Aubin
reviewboard/static/rb/js/resources/models/tests/fileAttachmentModelTests.es6.js
reviewboard/webapi/tests/test_file_attachment.py
reviewboard/webapi/tests/test_file_attachment_draft.py
Loading...