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

Don't depend on credentials to populate IPTV Manager data #266

Merged
merged 2 commits into from
Mar 8, 2021

Conversation

michaelarnauts
Copy link
Collaborator

On a clean install, when there are no credentials filled in, and IPTV Manager is querying for information, an exception is raised to indicate that there are no credentials.

We don't need to do this, since we can query the channels and EPG fine without credentials. Credentials will be asked when a channel or VOD is played for the first time.

@michaelarnauts michaelarnauts added the enhancement New feature or request label Mar 8, 2021
@dagwieers
Copy link
Collaborator

Similar problem with Yelo. On a default installation, if you enable IPTV Manager before adding credentials to Yelo, you get pop-ups from Yelo to add credentials. And it is very hard to get rid of it, especially if you happen to not have (working) credentials.

The use of IPTV Manager should not cause add-ons to give pop-up windows. I think we need to clarify this bit in our IPTV Manager documentation, and open an issue with the Yelo add-on. Providing an exception for this would be best, so the problem is visible and gets logged.

@michaelarnauts
Copy link
Collaborator Author

In this case, I don't need to send a token, so I can just query the EPG and Channel listing without credentials.

@michaelarnauts michaelarnauts merged commit f910d1d into master Mar 8, 2021
@michaelarnauts michaelarnauts deleted the no-errors-on-empy-config branch March 8, 2021 21:56
@michaelarnauts michaelarnauts added this to the v1.2.5 milestone Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants