search

Donovanphenry / BBFL

https://bbfl.vercel.app
0 stars 0 forks source link

Strengthen the method for getting the week number #26

Open Donovanphenry opened 2 months ago

Donovanphenry commented 2 months ago

ESPN seems to have discontinued support for the current week number in their API. We have implemented a band-aid for this week by hardcoding the value to 1 if the week number doesn't exist in their response. However, this is incredibly fragile, and no one wants to be in a position where they need to update a variable and re-deploy the whole website.

It would be more robust if we instead obtained this information by matching the current date against the dates in each week reference.

Donovanphenry commented 2 months ago

Hmm, well it looks to me like ESPN has added it back. We could still add this logic as a fallback though