search

pimatic / rfcontroljs

nodejs modul with protocol support for different 433mhz switches and weather stations for the RFControl Arduino library.
GNU General Public License v3.0
49 stars 54 forks source link

Changed battery status #19

Closed incmve closed 9 years ago

incmve commented 9 years ago

Instead of 0 and 1, good or bad battery.

incmve commented 9 years ago

The battery isn't in mobile-frontend yet is that right or did I make a boo-boo somewhere?

Icesory commented 9 years ago

The battery isn't shown in the mobile Frontend. I think this is not needed. And you should add the changes to the weather5.coffee too. The .js is the compiled version of the coffee file.

incmve commented 9 years ago

Thanks didn't know that. But how will you know when the battery is almost empty when there is nog visual feedback?

Icesory commented 9 years ago

I think a visual feedback isn't necessary. But it is possible to add a predicate handler so we can define a rule to log or push a message. But for this it is necessary to add a battery state to the HomeduinoRfWeatherDevice and all weather protocols must implement the battery state same.

incmve commented 9 years ago

Ok that sounds like a lot of work, I'm not a programmer so I can't realy say much about it. I am curious how it works so I might have a look at it later :)