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

Dev2 for enhancement of Hub V2 : add Temperature HumiditySensor #18

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

LGO44
Copy link
Contributor

@LGO44 LGO44 commented Oct 28, 2024

@msp1974: Thanks in advance for your help and advise.

  • enable notification of under or over power and WindowDoorSensor

WindowDoor. an issue on the turn_on or turn_off notification due to the id of the device given instead of the

  • create TemperatureHumiditySensor : threshold_sensor

problem: only the temperature is created not humidity ???

add attribute device_id for boiler_interface.

config_entry-wiser.json

@LGO44
Copy link
Contributor Author

LGO44 commented Oct 31, 2024

@msp1974 here is the last version of the API modified to support Threshold sensors

This type of sensor is the base for motion/light devices and also for the Merten CFMT2A devices

Binary sensor are also used for Waterleakage sensor and Motion Light sensor....

At this time, I have a problem with the commands :

request:
PATCH /data/v2/domain/BinarySensor/6 HTTP/1.1
SECRET: QDlDdH5woSQZm
{EnableNotification:None}

response:
{"Error":"Not Found",
"StatusCode":404}

cause :
PATCH /data/v2/domain/BinarySensor/6 : is the id of the device, this id has to be 1 ,id of the binary_sensor.

Thanks for your help Mark.

@LGO44
Copy link
Contributor Author

LGO44 commented Oct 31, 2024

The TemperatureHumidity sensor works fine with the climate. It generate 2 threshold sensors one for Temperature and the other for humidity...

I have an issue, because my code generate only one threshold _sensor (temperature):
my diag file
config_entry-wiser-01JBCRPKCEH147XTSNB9G1TKK0 (1).json

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.

1 participant