Prevent empty reviews from being published.

Review Request #13461 — Created Dec. 26, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

With the change to "Create Review" in the new unified banner, we made it
much easier to have an empty review get published. It used to be that
closing the review dialog with an empty review would delete the draft,
but we now keep it around.

This change adds a validation step for review publishing to ensure that
the review is not empty.

  • Attempted to publish an empty review from both the unified banner and
    the review dialog. Saw the expected error message.
  • Ran Python and JS unit tests.

Diff Revision 2 (Latest)

orig
1
2

Commits

First Last Summary ID Author
Prevent empty reviews from being published.
With the change to "Create Review" in the new unified banner, we made it much easier to have an empty review get published. It used to be that closing the review dialog with an empty review would delete the draft, but we now keep it around. This change adds a validation step for review publishing to ensure that the review is not empty. Testing Done: - Attempted to publish an empty review from both the unified banner and the review dialog. Saw the expected error message. - Ran Python and JS unit tests.
167042eaae2ab85ef40b58df792a04d4f8a6b78d David Trowbridge
reviewboard/notifications/tests/test_email_sending.py
reviewboard/reviews/models/review.py
reviewboard/static/rb/js/reviews/views/reviewDialogView.ts
Loading...