Toxblh / samsung-tv-control

📺 Remote control your TV via JS!
https://toxblh.github.io/samsung-tv-control/
MIT License
177 stars 34 forks source link

Unable to get Custom Input controls for Samsung UN60F8000 #33

Open wadehas opened 4 years ago

wadehas commented 4 years ago

Problem: Have installed Samsung-tvcontrol on Homebridge version 1.1.1 for Samsung UN60F8000 TV and unable to have custom inputs appear in accessory

Steps to Repro:

  1. Turn on Samsung UN60F8000
  2. Run "npx homebridge-samsungtv-control legacy 192.168.2.157 5C:F6:DC:04:C6:ED"
  3. Select "Allow" on resultant AllShare dialog that appears on TV. TV also mutes, which is expected and verifies command worked.
  4. Add the following JSON to config file for homebridge { "devices": [ { "name": "TV", "usn": "0a21fe80-00aa-1000-b88f-5cf6dcdefed8", "inputs": [ { "name": "Cable TV", "keys": "HDMI1" }, { "name": "Apple TV", "keys": "HDMI4" }, { "name": "XBox", "keys": "HDMI3" }, { "name": "BluRay", "keys": "HDMI2" } ], "disableUpnpSetters": true } ], "platform": "SamsungTVControl" }
  5. Restart Homebridge
  6. After Homebridge is back up, add accessory to Home App
  7. When TV is added to Home app, has the default inputs of "-" and "TV"
  8. Close Home App, and restart Homebridge
  9. After Hombridge is backup, open Home App and go to TV

Result: TV still has the default inputs, and does NOT have custom inputs specified in Config

Expected: Accessory should have custom inputs defined in config

Other Information: Setting the input to "TV" in the default inputs does change the input on the TV to "TV", and then accessory in HomeApp goes back to "-" which does nothing. Able to turn the TV off with the HomeApp. Can provide log if needed, only messages I have seen is that it detects the TV and states to add the TV to the Home App, even after it has been added.

mmende commented 4 years ago

Wrong repository 😉. Please post the issue here

Toxblh commented 4 years ago

wow, you use my lib! ~So if is it the same approach - you shouldn't use legacy, you should use "new" pair1 as I see in Readme. legacy it's for older TV.~ UPD: All good, should be legacy for 2013