forked from tbnobody/OpenDTU
-
Notifications
You must be signed in to change notification settings - Fork 66
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Feature: Set/obtain DPL upper power limit via MQTT
- Loading branch information
Showing
2 changed files
with
10 additions
and
1 deletion.
There are no files selected for viewing
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 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
1f6fdb7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @cerise21,
I saw this commit/change in the changelog of the newest release and directly wanted to use it.
Should this be automatically discovered by HomeAssistant?
Im not knowledgeable enough regarding the code, but I dont see any UpperPowerLimit in my MQTT devices in HomeAssistant.
1f6fdb7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ButterBetzi You are correct, we forgot to add the HASS auto-discovery for these new values. Please create a new issue (feature request) so we can track this.
1f6fdb7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created a very basic issue: #1178