-
Notifications
You must be signed in to change notification settings - Fork 155
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
Improvements for XTream API #217
Conversation
superolmo
commented
Feb 27, 2022
- Added method to gracefully handle connection errors during authentication and loading
- Added setting to not load adult content Hypnotix IPtv - Parental Control (hide channels?) #207 . This is only possible if the provider sets the is_adult flag for the relevant streams.
- Added sorting by stream name
- Added Waiting cursor while loading XTream streams
- Changed the handling of special characters in streams. Some streams were dropped.
- Changed print formatting
Hi can you rebase this please, off of the current master branch - there are a lots of conflicts, I think due to some of these commits being rather old - I could try to do it if you don't feel comfortable, but I can't really test it. Also, can you please squash down into one or two commits, and include this PR description in the commit message. Thanks |
@mtwebster If you do Squash and Merge you can get a single commit with the PR description in it, see https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges#squash-and-merge-your-pull-request-commits |
FYI, I have not yet test it. |
I tested. It is ready. |
Some folks don't like other people squashing their commits, which is why I ask first and offer to do it. |