Home Assistant grumpy about hostnames

rohrsh commented 2 years ago

On 2022.2:

Detected integration that accessed discovery_info.get('hostname') instead of discovery_info.hostname; this will fail in version 2022.6. Please report issue to the custom component author for my_velux using this method at custom_components/my_velux/config_flow.py, line 82: or not info.get("hostname")
Detected integration that accessed discovery_info['hostname'] instead of discovery_info.hostname; this will fail in version 2022.6. Please report issue to the custom component author for my_velux using this method at custom_components/my_velux/config_flow.py, line 83: or not info["hostname"].startswith("VELUX_KLF_LAN")
Detected integration that accessed discovery_info.get('host') instead of discovery_info.host; this will fail in version 2022.6. Please report issue to the custom component author for my_velux using this method at custom_components/my_velux/config_flow.py, line 87: self._host = info.get("host")

neildsb commented 2 years ago

+1 noted, holding back on 2022.5

pawlizio commented 2 years ago

This has been solved with the latest release 2.0

pawlizio / my_velux