device.status.level should be an int

Hexagon / node-telldus

Node bindings for telldus-core

Other

34 stars 10 forks source link

device.status.level should be an int #14

Closed nilzen closed 10 years ago

nilzen commented 10 years ago

device.status.level is a string but from what i can tell it's only used for dimmers and therefore it should be an int so we get a "correct" json response.

level: 64

instead of:

level: "64"

Hexagon commented 10 years ago

I think this commit will resolve this issue. I'm not sure though, as i have no dimmers to test it with :)

nilzen commented 10 years ago

Yep, seems to be correct now! :+1:

nilzen commented 10 years ago

Will you release a new version today with this, or should i patch the local file?

Hexagon commented 10 years ago

Publishing to npmjs right now!