Skip to content
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.

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

total_steps and total_distance for DailySteps should be able to be None #98

Closed
KopyWasTaken opened this issue Apr 3, 2025 · 0 comments

Comments

@KopyWasTaken
Copy link
Contributor

When I was pulling some of my data from the API, I was running into errors because I hadn't taken any steps yet for the day I was parsing (it was just past midnight). I think instead of this throwing an error in garth, it should be allowed, because it's a possibility that someone can have taken no steps and traveled no distance in the day yet.

Image

Those values were in fact accurate, and were successfully communicated through the Garmin API, so I don't think it should cause a garth error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant