Skip to content

Main doesn't warn when API data doesn't exist. #35

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
DrewWing opened this issue May 11, 2025 · 0 comments · May be fixed by #39
Open

Main doesn't warn when API data doesn't exist. #35

DrewWing opened this issue May 11, 2025 · 0 comments · May be fixed by #39
Assignees
@DrewWing
Labels
bug Something isn't working enhancement New feature or request
Milestone
v50.0

Comments

@DrewWing
Copy link
Owner

app/__main__.py should warn the user (or catch the resulting FileNotFoundError) when the API data doesn't exist.
Currently it just throws a FileNotFoundError when attempting to read the team numbers from team_list_filtered.csv.
@DrewWing DrewWing added bug Something isn't working enhancement New feature or request labels May 11, 2025
@DrewWing DrewWing added this to the v50.0 milestone May 11, 2025
@DrewWing DrewWing self-assigned this May 11, 2025
DrewWing added a commit that referenced this issue May 11, 2025
@DrewWing
Add critical logging if API data doesn't exist. Fix #35
3e89dae
@DrewWing DrewWing moved this from Backlog to Ready in EDrewcated Guesser by @DrewWing May 11, 2025
@DrewWing DrewWing linked a pull request May 13, 2025 that will close this issue
Feat/29 redo global calculations #39
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DrewWing DrewWing

Labels
bug Something isn't working enhancement New feature or request
Projects
EDrewcated Guesser by @DrewWing
Status: Ready
Milestone
v50.0
Development

Successfully merging a pull request may close this issue.

Feat/29 redo global calculations DrewWing/EDrewcatedGuesser
1 participant
@DrewWing