home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.68k stars 30.81k forks source link

LimitlessLED, bridge_led not showing up #6382

Closed JohanLeirnes closed 7 years ago

JohanLeirnes commented 7 years ago

Make sure you are running the latest version of Home Assistant before reporting an issue.

You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:

Home Assistant release (hass --version): 0.39.2

Python release (python3 --version): Python 3.4.3+

Component/platform: light/limitlessled

Description of problem: Bridge_led not showing up. All other groups show up and are controllable

Expected: Bridge_led showing up when "bridge_led: true"

Problem-relevant configuration.yaml entries and steps to reproduce:

light:
- platform: limitlessled
  bridges:
    - host: 192.168.1.**
      bridge_led: true
      groups:
      - number: 1
        name: ***
      - number: 2
        name: ***
      - number: 3
        name: ***
      - number: 4
        name: ***
janLo commented 7 years ago

If it worked in 0.38.x you need to specify port and version. See https://github.com/home-assistant/home-assistant/issues/6286 and https://github.com/home-assistant/home-assistant.github.io/pull/2171.

The version should be 5 and the port 8899.

JohanLeirnes commented 7 years ago

It did not. Its a v6 wifi ibox with a light inside it.

quadportnick commented 7 years ago

@bhaap I've derived a fix for this, please let me know if this helps and if so we'll get it committed. I don't know if bridge_led: true is actually necessary, will have to play with that next.

https://community.home-assistant.io/t/limitlessled-v6-bridge-led-control/12831/9?u=quadportnick

javydekoning commented 7 years ago

@quadportnick thanks, that seems to work.

quadportnick commented 7 years ago

Ok, I found that removing bridge_led has no impact on the light working with this method. Going to make the change.