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 3 and 4

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
Fixed line being longer than 80 characters.
d2c488279f3ed43a265e9f1195156b583951dec0 amalik2
Fixed publish being disabled when error occurs while saving review request fi...
380ddd8571c53a98797f281e55fbbd855f6b79d0 amalik2

Files

Loading...