search

edvaldeysteinsson / IKEA-Tradfri

IKEA Trådfri device type handler for SmartThings
64 stars 153 forks source link

Update ikea-tradfri.groovy #4

Closed brucerpdx closed 5 years ago

brucerpdx commented 6 years ago

Some sort of errant characters were causing pairing to fail.

edvaldeysteinsson commented 6 years ago

I' not sure how to solve this to be honest, the E27 bulbs i have actually have that name. I guess that IKEA accidentally named the early bulbs incorrectly and have fixed it in later versions but this change would result in the old ones not being added. Have you tried to just add the new ones and leave the old ones in? Do the new ones get added with the old and new names both being in the handler?

brucerpdx commented 6 years ago

What text encoding are you using? I'm getting just black question marks. I tried converting using Notepad++ but get gibberish.

edit; According to document.characterSet in my Chrome console, it's UTF-8 here on Github.

hkusulja commented 5 years ago

I also agree and confirm this small change, can you approve this PR / merge? @edvaldeysteinsson