You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Actually even if the devices returns a 200 for eureka_info, there is no guaranty that the device supports alarms or timers, we still would need to check assistant_supported. This can also come in handy in the future where we can check what other features is supported.
The check should probably be a combination of both like this:
Request eureka_info if 404 return device not compatible. Else continue check.
Check if assistant_supported is true, return device is compatible. If false return device is not compatible.
If eureka_info is 200, we should create device sensor. If assistant_supported is true we should also create alarms and timers sensors and do not disturb switch.
Actually even if the devices returns a
200
foreureka_info
, there is no guaranty that the device supports alarms or timers, we still would need to checkassistant_supported
. This can also come in handy in the future where we can check what other features is supported.The check should probably be a combination of both like this:
eureka_info
if404
return device not compatible. Else continue check.assistant_supported
is true, return device is compatible. If false return device is not compatible.Originally posted by @DurgNomis-drol in #212 (comment)
eureka_info
example:The text was updated successfully, but these errors were encountered: