[NEW] Add support to Headers & Loaders to the View Model & much more tests #2263
[NEW] Add support to Headers & Loaders to the View Model & much more tests #2263
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some files could not be reviewed due to errors:
Loading configuration from '.swiftlint.yml'
Loading configuration from '.swiftlint.yml' Linting Swift files at paths Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift No lintable files found at paths: 'Rocket.Chat/Views/Chat/New, Chat/Cells/UnreadMarkerCell.swift'
var containsDateSeparator: Bool { return daySeparator != nil } | ||
|
||
init(message: UnmanagedMessage, daySeparator: Date? = nil, sequential: Bool = false, unreadIndicator: Bool = false) { | ||
init(message: UnmanagedMessage, daySeparator: Date? = nil, sequential: Bool = false, unreadIndicator: Bool = false, loader: Bool = false, header: Bool = false) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line Length Violation: Line should be 120 characters or less: currently 165 characters (line_length)
Codecov Report
@@ Coverage Diff @@
## feature/integrate-rc-view-controller #2263 +/- ##
========================================================================
+ Coverage 25.73% 25.82% +0.08%
========================================================================
Files 434 436 +2
Lines 16341 16367 +26
========================================================================
+ Hits 4206 4226 +20
- Misses 12135 12141 +6
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! 👍
@RocketChat/ios
First needs to merge #2262