search

notoriousbdg / Home-AssistantConfig

Apache License 2.0
180 stars 49 forks source link

The battery_alert package has trouble with 0.170 HA release due to Groups changes #25

Closed luckyjay closed 3 years ago

luckyjay commented 4 years ago

Check out breaking changes for 0.107 https://www.home-assistant.io/blog/2020/03/18/release-107/ and the note related to Groups. Because the battery_alert package uses group.set, it's having some issues with this new release because that service no longer exists - at least, not with that name.

Not sure how to fix, but wanted to report the issue.

Error returned by HA is "Unable to find service group/set."

iMiMx commented 4 years ago

https://www.home-assistant.io/integrations/group/

Lines such as 'view' and 'control' in the battery_alert.yaml packages file are no longer valid, for example:

battery_alert: control: hidden

battery_view: view: yes

iMiMx commented 4 years ago

Seems like there is a pull request already that fixes this:

https://github.com/notoriousbdg/Home-AssistantConfig/pull/23/files

IsaTTeN commented 4 years ago

I have the same problem

lpt2007 commented 4 years ago

Same problem here. Can someone update script to work 0.107+?

It is good script. It should be nice to see it in HASS COMMUNITY STORE ;)

idfxken commented 4 years ago

yeah, i made a merge request but think notorious is busy with other life stuff atm. you can just use an editor to find and delete the offending lines. Edit: To be specific, find any line containing:

view: or control:

schumi2004 commented 4 years ago

Not only does the group/view give issues but it also returns the following error for me when I toggle settings: Unable to find service input_number/turn_on Home Assistant 0.110.3

KptnKMan commented 3 years ago

Hi everyone, I got this to work with my HassOS install. Its tested with HA v0.117.3 at this time, so should work.

I made a PR here and added a little documentation: https://github.com/notoriousbdg/Home-AssistantConfig/pull/27

Feel free to test, the direct raw file is here: https://raw.githubusercontent.com/notoriousbdg/Home-AssistantConfig/164d3d736abfed943c4a4a71ec769022dc3045aa/packages/battery_alert.yaml

I'm interested to know if this works for others.

Thanks.

notoriousbdg commented 3 years ago

https://github.com/notoriousbdg/Home-AssistantConfig/pull/27 has been merged. Thanks @KptnKMan!