Barma-lej / halandroid

Worx Landroid package for Home Assistant based on Landroid Cloud by @MTrab
MIT License
112 stars 49 forks source link

Dependency of sensor.date_time #52

Closed Mincka closed 1 year ago

Mincka commented 2 years ago

Hello, Thank you for this great package. I discovered through the error notification that there is a requirement of Time & Date integration with date_time otherwise the user will get a "- unknown" at the end of the message for mower_notify_status and mower_notify_error automations.

The sensor is referenced here: https://github.com/Barma-lej/halandroid/blob/master/packages/landroid.yaml#L491 and here: https://github.com/Barma-lej/halandroid/blob/master/packages/landroid.yaml#L491

I am not sure if that's really mandatory to have it here in the first place since the notification has already a time. I leave it up to you if you prefer to remove the dependency or add the "Time & Date" integration as a requirement in the README.md.

Mincka commented 1 year ago

Thanks @CSlush!