add support for event entities #644
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
this commit allows to migrate from z2m legacy triggers to experimental event entities
|
Hey @xeaon, thank you so much for your contribution! 🚀 🔄 We're currently running a few checks to make sure that everything is great with your contribution. Results are coming soon, stay tuned! |
|
fixes #622 |
LaStrada
reviewed
Jan 10, 2025
| # only for zigbee2mqtt, check if the event is relative to a real state change, and not only some minor changes in the sensor attributes | ||
| # this is required since multiple state_changed events are fired for a single button press, with the result of the automation being triggered multiple times | ||
| - '{{ integration_id != "zigbee2mqtt" or trigger.event.data.new_state.state != trigger.event.data.old_state.state }}' | ||
| # - '{{ integration_id != "zigbee2mqtt" or trigger.event.data.new_state.attributes.event_type != trigger.event.data.old_state.attributes.event_type }}' |
There was a problem hiding this comment.
Why was this commented out? Isn't this needed anymore?
There was a problem hiding this comment.
afaict there is no need for this line. i'm testing this for about a week now and everything works as expected
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Thank you for taking the time to work on a Pull Request. Your contribution is really appreciated! 🎉
Please don't delete any part of the template, since keeping the provided structure will help maintainers to review your work more rapidly.
Sections marked as * are required and need to be filled in.
Breaking change
This change requires you to enable
Home Assistant experimental event entitiesin Zigbee2MQTT. Enabling this experimental features may break you current automations, dashboards, etc.Enabling event entities
Event entities in HA
Trace of working automation
Proposed change*
This change allows to make use of event entities in HA. Enabling
Home Assistant experimental event entitiesin Zigbee2MQTT will add event entities to your devices, like the ikea_e1744, which we can utilize to define the Controller Entity in this blueprint.Checklist*
npm run formatbefore submitting my Pull Request.