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

Update automoli.py #91

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Update automoli.py #91

merged 1 commit into from
Jan 18, 2022

Conversation

Allthebester
Copy link
Contributor

Fix for AppDaemon 4.2.0, update "entity" to "entity_id" for "set_state" and "listen_state"

Fix for AppDaemon 4.2.0, update "entity" to "entity_id" for "set_state" and "listen_state"
@ALutz273
Copy link

This commit works fine!

Copy link

@FredericESH FredericESH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works fine

Copy link

@FredericESH FredericESH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works fine now

@legovaer
Copy link
Contributor

Thanks a lot for this! As I only started using this app after the 4.2.0 release, I started to think that I was going mental. I've spent hours on reading the documentation and ended up on modifying the entire script.

Life saver!

@gogui63
Copy link

gogui63 commented Jan 17, 2022

@benleb Could you merge this PR ?

@benleb
Copy link
Owner

benleb commented Jan 17, 2022

sorry, give me a few hours 😄✌️

@benleb benleb merged commit 3c2c6fe into benleb:master Jan 18, 2022
@gogui63
Copy link

gogui63 commented Jan 18, 2022

@benleb Don't forget to make a new release when you have 5 minutes to get update on HACS side 😃
And thanks again for this HA plugin !

@benleb
Copy link
Owner

benleb commented Jan 20, 2022

tomorrow! 👍

@thundergreen
Copy link
Contributor

WOW! thanks I was really confused that my lights would randomly turn on and off after Appdaemon update :P Now I know the answer. Thanks a thousand times!

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

Successfully merging this pull request may close these issues.

7 participants