Skip to content
This repository has been archived by the owner on Oct 1, 2021. It is now read-only.

Add Yeelight Ceiling Light support #168

Merged
merged 1 commit into from
Feb 4, 2018

Conversation

pauln
Copy link
Contributor

@pauln pauln commented Feb 3, 2018

Adds discovery of Yeelight Ceiling Light (Wifi + Bluetooth).

Adds discovery of Yeelight Ceiling Light (Wifi + Bluetooth).
@rytilahti
Copy link
Contributor

Have you tested this on your own system? There used to be a check for it (albeit with a bit different name), which got reverted in #152.

@pauln
Copy link
Contributor Author

pauln commented Feb 4, 2018

@rytilahti Although (in line with the comments on #151) my ceiling lamp never used to announce itself over mDNS, it seems to have recently begun doing so (I'm currently on firmware v1.5.5_0178, which also added support for the ceiling lamp acting as a wifi gateway for BLE devices). As such, I'm now getting periodic log messages in Home Assistant about it:
Unknown miio device found: ServiceInfo(type='_miio._udp.local.', name='yeelink-light-ceiling1_mibtXXXXXXXX._miio._udp.local.' [clipped for brevity]

Although I hadn't earlier, I've just tested this change and can confirm that it works (for me) - Home Assistant discovers the lamp and can control it with no manual configuration, whereas I've previously had to add it to configuration.yaml to get Home Assistant to see it. (It still works with the manual config too, so it's a non-breaking change as far as I can tell.)

@rytilahti
Copy link
Contributor

Ok, let's merge this then (the linter errors are not caused by this PR), great to see they have fixed the discovery :-)

@rytilahti rytilahti merged commit dce9564 into home-assistant-libs:master Feb 4, 2018
@Notrial
Copy link

Notrial commented Feb 12, 2018

Hi guys,
I've updated to 0.63 and this problem seems to remain. I'm new to HA, so I'm unfamiliar with component updates and whether the fixes are applied automatically with the next HA build? Do I have to do something manually in order to fix this?

@MartinHjelmare
Copy link

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels

Merged PRs should not be used for support or bug reports. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants