I'm thinking, now, that anything UI related has no business even instantiating an NflApi class, and that we need to just interact with the model, and that there is an offline job that pulls the current info into the DB on a schedule, maybe once every 10 minutes or so. So, here's my proposal:

new table in the DB for week_info which has season, week_no and season_type` ints
New 'job' that gets run by the scheduler to fetch the current_info from the API every (say) 10 minutes
new method to replace the WeekInfo get_current_info to just get it from the DB
Ask claude if there is any way for me to 'assert' (sentry log) if the NflApi class is initted within the context of a FastApi route
Decommission the 'old' week_info

That should effectively decouple the UI from the API

I'm thinking, now, that anything UI related has no business even instantiating an NflApi class, and that we need to *just* interact with the model, and that there is an offline job that pulls the current info into the DB on a schedule, maybe once every 10 minutes or so. So, here's my proposal: - [ ] new table in the DB for `week_info` which has season, week_no and season_type` ints - [ ] New 'job' that gets run by the scheduler to fetch the current_info from the API every (say) 10 minutes - [ ] new method to replace the WeekInfo get_current_info to just get it from the DB - [ ] Ask claude if there is any way for me to 'assert' (sentry log) if the NflApi class is initted within the context of a FastApi route - [ ] Decommission the 'old' week_info That should effectively decouple the UI from the API

johnsturgeon referenced this issue from a commit

2026-08-21 17:29:14 +02:00

[skip review] This does technically eliminate the call to the api (read notes)

johnsturgeon closed this issue

2026-08-21 17:29:14 +02:00

johnsturgeon referenced this issue from a pull request that will close it,

2026-08-21 17:29:28 +02:00

[skip review] This does technically eliminate the call to the api (read notes) #405

johnsturgeon referenced this issue from a commit

2026-08-21 17:46:44 +02:00

[skip review] This does technically eliminate the call to the api (read notes) (#405)