New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CIS-295] Channels pagination #438
Conversation
Codecov Report
@@ Coverage Diff @@
## master #438 +/- ##
==========================================
- Coverage 84.19% 84.15% -0.04%
==========================================
Files 94 94
Lines 3536 3541 +5
==========================================
+ Hits 2977 2980 +3
- Misses 559 561 +2
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.
Nice and easy 👍 Thanks
controller.loadNextChannels(limit: limit) { [callbackQueueID] error in | ||
AssertTestQueue(withId: callbackQueueID) | ||
XCTAssertNil(error) | ||
completionCalled = true | ||
} |
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.
Smells like we should create some abstraction for this boilerplate, too. It's everywhere.
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.
Probably yes. Will think about it when will touch this again :)
In this PR:
ChannelListController
SampleApp
Q: I’m not doing any checks for empty
channels
inloadNextChannels
function cause it’s not an issue to have 0 offset. But there is a possibility that this function will be called successfully beforestartUpdating
call. (channels
has empty array as fallback value) If we want to restrict this we can check for controllerstate
but I’m not sure if we should do that.