AdyRock / com.ady.button_plus

GNU General Public License v3.0
2 stars 3 forks source link

Setting right button toplabel fails #77

Closed Joolee closed 8 months ago

Joolee commented 8 months ago

Simple bug: https://github.com/AdyRock/com.ady.button_plus/blob/f6a691039c5867131176193c945f7117ecfc3895/drivers/panel_hardware/device.js#L262C71-L262C71 contains a / too many after ${id}

Also, the logic on line 250, 259, 299 and 309 does not seem to work as I cannot change the label of a button that is bound to a device. I don't think the mqtt message should contain the device ID but I haven't tested this. and have not confirmed the bug extensively.