search

house-of-abbey / GarminHomeAssistant

Garmin application to provide a dashboard to control your Home Assistant
https://community.home-assistant.io/t/home-assistant-app-for-garmin/637348
MIT License
101 stars 14 forks source link

Bug fix for activity reporting #167

Closed philipabbey closed 2 months ago

philipabbey commented 2 months ago

Added 'has' clauses around additional ActivityMonitor.Info fields that are not present on all devices. I have also checked the other reporting fields to ensure they are not device dependent.

philipabbey commented 2 months ago

@KPWhiver I'm inviting you to the team, when you've accepted the invitation I'll add you as a reviewer for this issue.

philipabbey commented 2 months ago

To fix this bug, you'll also have to add has clauses to the background delegate.

Goes to show how flakey testing can be on these development tools. I have now invoked the background service from the simulation menu.

philipabbey commented 2 months ago

@KPWhiver Thank you for your assistance with this amendment.