Use automergeAttrs for BaseResource and subclasses.
Review Request #13853 — Created May 13, 2024 and submitted — Latest diff uploaded
This change makes use of
automergeAttrs
for our resource classes.
Spina itself specifies this fordefaults
, which means we don't have to
manually call_.defaults()
for all of these. This additionally makes
use of it forattrToJsonMap
,deserializers
, andserializers
.
- Ran js-tests.
- Verified that all objects were properly automerged.
Diff Revision 1
This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.
orig
1
2
Commits
Files
reviewboard/static/rb/js/common/resources/models/baseCommentModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/baseResourceModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/draftReviewModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/fileAttachmentModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/reviewModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/reviewReplyModel.ts |
---|
reviewboard/static/rb/js/common/resources/models/reviewRequestModel.ts |
---|