Remove long-deprecated callbacks style resource invocations.
Review Request #14158 — Created Sept. 11, 2024 and updated — Latest diff uploaded
In Review Board 5, we added promises for our asynchronous operations in
our resource classes. We did this with fallbacks to handle old-style
calls that passed in callback methods. This change converts those
fallbacks to be errors instead.
Ran js-tests.
Changes between revision 1 and 2
orig
1
2
3
Commits
Files
| reviewboard/static/rb/js/common/collections/baseCollection.ts |
|---|
| reviewboard/static/rb/js/common/resources/collections/repositoryCommitsCollection.ts |
|---|
| reviewboard/static/rb/js/common/resources/collections/resourceCollection.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/baseResourceModel.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/draftResourceChildModelMixin.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/draftResourceModelMixin.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/draftReviewModel.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/draftReviewRequestModel.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/reviewGroupModel.ts |
|---|
| reviewboard/static/rb/js/common/resources/models/reviewReplyModel.ts |
|---|
| reviewboard/static/rb/js/reviews/models/commentEditorModel.ts |
|---|