zacs / ha-nfl

NFL game scores custom component for Home Assistant
43 stars 32 forks source link

Error fetching Browns data: list index out of range #16

Closed Jefe2 closed 2 years ago

Jefe2 commented 2 years ago

The API apparently changed from last season. I have three NFL integrations installed and they worked very well last season. This week, the Steelers-Seahawks game is showing up on one card, but I’m getting an error for the other 2 integrations I have installed:

Error fetching Ravens data: list index out of range
Error fetching Browns data: list index out of range

There’s no hurry in taking a look, it’s only pre-season 😉 Thanks!

gonzotek commented 2 years ago

I think it's probably the same issue I found yesterday: https://github.com/zacs/ha-nfl/pull/15 I put in a pull request for a fix and @zacs accepted it but he'll need to do a new release for hacs to pick it up, I think. I logged another new issue, https://github.com/zacs/ha-nfl/issues/14 from last weeks delayed game, that neither he nor I have gotten around to fixing yet, so he may be holding until he can (or someone) has the time to fix as well. If you're comfortable editing python files, you can look at the pull request and make the change to your local install pretty easily (see the files changed tab in the PR) until there's a new release.

Jefe2 commented 2 years ago

Many thanks. I applied the change you suggested and it works well. All three teams/games are displayed now... And I learned something.

zacs commented 2 years ago

Cutting the release now, just wanted to see if there were any more bugfixes to include, but seems mostly quiet 🤞.