Skip to content
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

Banned channels breaking infinite scroll mechanism #466

Closed
Robby- opened this issue Jun 1, 2017 · 6 comments
Closed

Banned channels breaking infinite scroll mechanism #466

Robby- opened this issue Jun 1, 2017 · 6 comments
Labels

Comments

@Robby-
Copy link

Robby- commented Jun 1, 2017

Description

I can't see more than 21 followed channels. However, at the top, next to Followed Channels it says the correct number: (60). There is also no Loading message or Fetch more button appearing at the bottom.
I also looked at the logfiles that are produced but there's nothing useful in them, even with --loglevel debug.

Expected / Actual behavior

Expected: I should be able to see the full list of followed channels.
Actual: Only 21 channels are shown.

Reproduction steps

Under My, click on Channels.

Environment details (operating system, etc.)

Windows 7 SP1, 64-bit
Streamlink Twitch GUI 1.3.1

@bastimeyer
Copy link
Member

bastimeyer commented Jun 2, 2017

Is it possible that one of your followed channels is banned? There was a similar issue a couple of months ago in the subscriptions list.

The reason why I'm asking is because of the implementation of the infinite scroll logic here. It's not making use of the _total metadata returned by Twitch API responses. The reason for this is that certain API calls won't return this metadata, so it's not possible to know the total amount of records of all routes. What the InfiniteScrollMixin does is comparing the returned number of records with the requested number. When the returned number is lower than the requested one, this usually means that the end of the list has been reached.

That's why I'm asking about banned channels. Twitch seems to just leave out banned channels in the API responses, instead of filling the missing slots with additional channels.

This issue could be fixed by making use of metadata, but as I've said, this is inconsistent. But yeah, not being able to browse the whole list is definitely an issue.

@bastimeyer bastimeyer added the bug label Jun 2, 2017
@Robby-
Copy link
Author

Robby- commented Jun 2, 2017

Yes, there is indeed one banned channel that I know of.

@Robby-
Copy link
Author

Robby- commented Jun 2, 2017

Would it be possible to unfollow banned channels if one knows its name? Or does the API not allow for such a thing (and thus are stuck with this)? If it is possible I'll leave it like it is for now, though, for the purposes of this bug report.

@bastimeyer bastimeyer changed the title Can't see more than 21 followed channels Banned channels breaking infinite scroll mechanism Jun 4, 2017
@bastimeyer
Copy link
Member

As long as you're able to retrieve data from the API, you're able to unfollow.

@Robby-
Copy link
Author

Robby- commented Jun 4, 2017

Ok, cool. I know Chatty has an /unfollow <channel> command and I was hoping to do it that way.

@bastimeyer
Copy link
Member

@Robby-
Could you please see if the changes are working for you? This should be covered by the added tests, but I'd like to know if I haven't missed anything here. I don't have any banned channels in my following list, so I can't test this myself and need some help. Thanks!

The changes are on the infinite-scroll branch:
ac04bfa / master...infinite-scroll

Build instructions:
https://github.com/streamlink/streamlink-twitch-gui/blob/v1.3.1/CONTRIBUTING.md#developing-and-building

git clone -b infinite-scroll https://github.com/streamlink/streamlink-twitch-gui
cd streamlink-twitch-gui
npm install -g bower grunt-cli
npm install
grunt release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants