split favorite channels to own accessor, adds watch next channel to t…#469
Conversation
nielsvanvelzen
left a comment
There was a problem hiding this comment.
Overall this looks good, I think it makes more sense to add a new "getWatchNextPrograms()" function to ChannelRepository, that is similar to "getProgramsByChannel()" but uses "ChannelResolver.CHANNEL_ID_WATCH_NEXT".
That way you don't need to query all the channels just to pick the one we already know the id of.
62f3687 to
4efd6ac
Compare
|
@nielsvanvelzen , hope this looks better 👌 |
|
I meant to query the programs directly and skip querying the channel, sorry if that wasn't clear! I've added an additional commit to make it behave that way now. Thanks for the contribution! |
Understandably, since the code is new to me, but that was cleaner code 👍 |
Uh oh!
There was an error while loading. Please reload this page.