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.

Diff Revision 4 (Latest)

orig
1
2
3
4

Commits

First Last Summary ID Author
Fixed publish being disabled when error occurs while saving review request fi...
380ddd8571c53a98797f281e55fbbd855f6b79d0 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
reviewboard/static/rb/js/views/tests/reviewRequestEditorViewTests.es6.js
Loading...