Convert BaseResource.save to return a promise.

Review Request #11642 — Created June 2, 2021 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

This change updates the save machinery for various resources to use
promises and async/await instead of callbacks.

  • Ran js-tests.
  • Manually tested API token creation, comment editing, review request
    field editing, post-commit review request creation, ship-it revocation,
    and file attachment uploading.

Changes between revision 4 and 5

orig
1
2
3
4
5

Commits

Summary ID Author
Convert BaseResource.save to return a promise.
This change updates the save machinery for various resources to use promises and async/await instead of callbacks. Testing Done: - Ran js-tests. - Manually tested API token creation, comment editing, review request field editing, post-commit review request creation, ship-it revocation, and file attachment uploading.
3a7c5d1c2b30161af04cc48196b0a352156dac62 David Trowbridge
Convert BaseResource.save to return a promise.
This change updates the save machinery for various resources to use promises and async/await instead of callbacks. Testing Done: - Ran js-tests. - Manually tested API token creation, comment editing, review request field editing, post-commit review request creation, ship-it revocation, and file attachment uploading.
fe13a59893bbf8cdbb32a68242c9a9926c5d7ec9 David Trowbridge
reviewboard/static/rb/js/models/tests/commentEditorModelTests.es6.js
reviewboard/static/rb/js/models/tests/reviewRequestEditorModelTests.es6.js
reviewboard/static/rb/js/newReviewRequest/views/tests/postCommitViewTests.es6.js
reviewboard/static/rb/js/pages/views/tests/reviewablePageViewTests.es6.js
reviewboard/static/rb/js/resources/models/draftReviewRequestModel.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/reviewRequestPage/models/tests/reviewReplyEditorModelTests.es6.js
reviewboard/static/rb/js/views/tests/commentDialogViewTests.es6.js
reviewboard/static/rb/js/views/tests/reviewDialogViewTests.es6.js
reviewboard/static/rb/js/views/tests/reviewRequestEditorViewTests.es6.js
reviewboard/static/rb/js/views/tests/reviewRequestFieldViewsTests.es6.js
Loading...