search

daviswieck / ha-college-football

5 stars 1 forks source link

NOT_FOUND incorrectly showing for several different schools #4

Closed ThatTallGuy21 closed 1 year ago

ThatTallGuy21 commented 1 year ago

I've made 5 attempts at getting a school to display and have only gotten it to work once with TEX. For TEX, the sensor state shows correctly as POST. However, I'm getting a sensor state of NOT_FOUND for ALA, ND, MICH, and PSU despite all of them playing recently and in the top 25.

daviswieck commented 1 year ago

Change the api endpoint in the Const file to https://site.api.espn.com/apis/site/v2/sports/football/college-football/scoreboard

This is for top 25 teams.

ThatTallGuy21 commented 1 year ago

Ah, that did the trick! I missed that. Thank you

Exit2Studios commented 1 year ago

I realize this is closed, but I'm unable to get anything to show. Is there a better way to add multiple teams than using the Integrations UI in HA?

daviswieck commented 1 year ago

In your config file add: Change teams to whatever you want them to. 

sensor:
  - platform: college_football
    team_id: 'TTU'

  - platform: college_football
    team_id: 'OKST'
Exit2Studios commented 1 year ago

Got it. What if they’re not in the same conference and one is not top 25?

daviswieck commented 1 year ago

In the Readme, it has instructions on how to change the API per conference along with the conference ID number.

Exit2Studios commented 1 year ago

I saw that, but unclear on how to follow two teams that are in different conferences (and not top 25).

daviswieck commented 1 year ago

I see. That is a problem at the moment. You can run a limit on all fbs teams, but it breaks the code at the moment.

https://site.api.espn.com/apis/site/v2/sports/football/college-football/scoreboard?groups=80&limit=150

daviswieck commented 1 year ago

Well it actually didn't break for me when I tried it this time.... Just pushed an update. Let me know if you have any problems.

https://github.com/daviswieck/ha-college-football/releases/tag/V1.0

Exit2Studios commented 1 year ago

Interestingly TEX and ALA are both showing up, so I guess Texas is top 25 afterall lol.

image