Fixed disabled buttons if an error happens updating a review field.

Review Request #10689 — Created Sept. 6, 2019 and updated — Latest diff uploaded

Information

Review Board
master

Reviewers

The publish button ends up being disabled while the changes made to the
model are being saved, so that the user does not repeatedly click the
publish button. If the request fails due to an error with one of the
fields that were changed, then the publish button does not ever end up
being re-enabled. Now, the model emits an event when an error happens
in the above case, and the view listens for that event being published.
When the view is notified of that event, it re-enables the buttons.

I tested this locally by following the instructions listed on the bug
report page (bug #4777).

I also ran the JavaScript unit tests.

Changes between revision 1 and 2

orig
1
2
3
4

Commits

Summary ID Author
Fixed publish button being disabled when an invalid user or group is specified.
1dfaa19bb9d726c25eebbf618862538ff2241328 amalik2
Changed ocurrences of double-quotes to single-quotes.
8d241e8dafd90a60b3ce0f6b08b46f1313270c67 amalik2
reviewboard/static/rb/js/models/reviewRequestEditorModel.es6.js
reviewboard/static/rb/js/models/tests/reviewRequestEditorModelTests.es6.js
reviewboard/static/rb/js/views/reviewRequestEditorView.es6.js
Loading...