Dilbert66 / esphome-vistaECP

This is an implementation of an ESPHOME custom component and ESP Library to interface directly to a Safewatch/Honeywell/Ademco Vista 15/20 alarm system using the ECP interface and very inexpensive ESP8266/ESP32 modules .
GNU Lesser General Public License v2.1
125 stars 21 forks source link

rf decoding #20

Closed nitej closed 3 years ago

nitej commented 3 years ago

found the bad curly brackets. This should place the decoding of extcmd in the right place

Dilbert66 commented 3 years ago

Can you do full PR. I reverted your last PR and cleaned up a few things on my end so lost your previous changes. If you do a clean PR this should fix it.

nitej commented 3 years ago

Yes, let me put a new PR together

Dilbert66 commented 3 years ago

One other thing, this is backwards. sprintf(vista.extcmd[5], "%02x", rf_serial_char); should be sprintf(rf_serial_char, "%02x",vista.extcmd[5]);