search

eracknaphobia / plugin.video.nhlgcl

Watch NHL.tv in KODI
GNU General Public License v2.0
25 stars 17 forks source link

not showing all stream options #53

Closed bellt closed 6 months ago

bellt commented 6 months ago

With nhltv not always naming the streams "home" or "away", we sometimes end up with some stream options missing from the list. My changes, included in 17d3075, assume that there would be only one stream option that had a name different to "home" or "away", but further investigation has found that there are sometimes multiple other stream names, as shown in these examples from the website image image image

Within this plugin, with the above examples the user wouldn't be presented with all of the available streams. I think we would have had a similar issue before the recent changes.

I am currently working on a fix for this, and will make a pull request when I've finished it.

eracknaphobia commented 6 months ago

I do like the dictionary approach as it is scalable.