Convert ready operations to promises.

Review Request #11842 — Created Oct. 4, 2021 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

This change updates the ready method to use promises and async/await
instead of callbacks. This flattens almost all of our remaining
asynchronous operations.

  • Ran js-tests.
  • Smoke tested various UI functionality.

Diff Revision 3 (Latest)

orig
1
2
3

Commits

First Last Summary ID Author
Convert ready operations to promises.
This change updates the `ready` method to use promises and async/await instead of callbacks. This flattens almost all of our remaining asynchronous operations. Testing Done: - Ran js-tests. - Smoke tested various UI functionality.
cc05130a062500c31152c7eb5ef99746ba141023 David Trowbridge
This diff has been split across 2 pages: 1 2 >
reviewboard/static/rb/js/accountPrefsPage/views/apiTokensView.es6.js
reviewboard/static/rb/js/models/commentEditorModel.es6.js
reviewboard/static/rb/js/models/commentIssueManagerModel.es6.js
reviewboard/static/rb/js/models/regionCommentBlockModel.es6.js
reviewboard/static/rb/js/pages/models/reviewablePageModel.es6.js
reviewboard/static/rb/js/pages/models/tests/reviewablePageModelTests.es6.js
reviewboard/static/rb/js/pages/views/tests/diffViewerPageViewTests.es6.js
reviewboard/static/rb/js/pages/views/tests/reviewablePageViewTests.es6.js
reviewboard/static/rb/js/resources/collections/resourceCollection.es6.js
reviewboard/static/rb/js/resources/collections/tests/resourceCollectionTests.es6.js
reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js
reviewboard/static/rb/js/resources/models/draftResourceChildModelMixin.es6.js
reviewboard/static/rb/js/resources/models/draftResourceModelMixin.es6.js
reviewboard/static/rb/js/resources/models/draftReviewModel.es6.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.es6.js
reviewboard/static/rb/js/resources/models/reviewReplyModel.es6.js
reviewboard/static/rb/js/resources/models/reviewRequestModel.es6.js
reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js
reviewboard/static/rb/js/resources/models/tests/draftReviewModelTests.es6.js
reviewboard/static/rb/js/resources/models/tests/draftReviewRequestModelTests.es6.js
This diff has been split across 2 pages: 1 2 >
Loading...