Clean up comment block and reviewable models
Review Request #10191 — Created Oct. 3, 2018 and submitted
The
RB.AbstractCommentBlock
andRB.AbstractReviewable
heirarchies
needed some cleaning up. Most of them were lacking documentation about
what their model attributes were. Also theRB.DiffCommentBlock
model
was using ES5 stylemethod: function() {...}
definitions instead of
ES6 stylemethod() {...}
definitions. Finally, the
RB.FileAttachmentReviewable
model was re-defining thecaption
attribute which it inherits from its parent model.All of these issues have been cleaned up. All model attributes are
documented with links back to the parent models for additional
attributes.
Ran JS tests.
Description | From | Last Updated |
---|---|---|
Testing done? |
david | |
Typo in the description's first line: "heirarchies" -> "hierarchies" |
chipx86 | |
I don't think we need to say "The jQuery object for ...", we can just say "The first row in … |
david | |
Same here. |
david | |
Can we alphabetise these? |
david | |
Two periods. |
david | |
typo: objecting? |
david | |
Instead of "differ" (which is confusing with "diffed/diffs"), let's say "whether the attachments have different review UI types" |
david | |
typo: textcurrently |
david | |
Same comment about "the jquery object" |
david | |
And here |
david | |
Missing definitions for these. |
david | |
"ID" |
chipx86 | |
:js:class: |
chipx86 | |
Array |
chipx86 | |
Array |
chipx86 | |
Array |
chipx86 | |
:js:class: |
chipx86 | |
Too many blank lines. |
chipx86 | |
:js:class: |
chipx86 |
- Change Summary:
-
Addressed feedback
- Testing Done:
-
+ Ran JS tests.
- Commit:
-
0c14bb0f88a9fc5edd9a399e06e4b8184dd006d1d0c9dffa77b52a570c704ac005c6244b0cba6b43
- Diff:
-
Revision 2 (+209 -3)
Checks run (2 succeeded)
- Change Summary:
-
Fix depends-on relationship.
- Change Summary:
-
Addressed feedback
- Commit:
-
d0c9dffa77b52a570c704ac005c6244b0cba6b43787f27729eb6aef6de6e7c865e76a5dab5406fd8
- Diff:
-
Revision 3 (+208 -3)