Sholofly / lghorizon

Custom integration to control LG Horizon settop boxes for Ziggo(NL), Magenta(AT), UPC(CH), Virgin(GB, IE), Telenet(BE)
GNU General Public License v3.0
62 stars 15 forks source link

Virgin Media (GB) didn't connect without a value in identifier field #30

Closed callumeveratt closed 1 year ago

callumeveratt commented 1 year ago

The documentation appears to say that the identifier field is not required unless using Telenet.

I'm just moving from the old Arris integration and when I tried to add originally I saw this error in HA:

Logger: custom_components.lghorizon.config_flow Source: custom_components/lghorizon/config_flow.py:78 Integration: LG Horizon (documentation, issues) First occurred: 10:36:38 (3 occurrences) Last logged: 10:36:55 'indentifier'

I went back and added the integration but put 'no' in the identifier field (just as a test) and the integration added and it all works. So not sure if the config flow expects a value for identifier?

Sholofly commented 1 year ago

@callumeveratt What version did you install?

callumeveratt commented 1 year ago

Literally just installed it so grabbed the most recent at the time - v0.3.10

Sholofly commented 1 year ago

Think it's abut a typo on my side. I'll fix it rightaway