PicoW_HomeAssistant_Starter icon indicating copy to clipboard operation
PicoW_HomeAssistant_Starter copied to clipboard

Suggestion: Add note in HAIntegration.cpp

Open JacobChrist opened this issue 2 years ago • 3 comments
trafficstars

Suggestion: Add note in HAIntegration.cpp that HASwitch(uniqueid) must not contain spaces.

HASwitch led("led");  // unique identifier must not contain spaces

JacobChrist avatar May 21 '23 05:05 JacobChrist

Nice catch! I'm open to a pull request to this effect.

daniloc avatar May 21 '23 18:05 daniloc

While looking around at the arduino-home-assistant examples I notices that you can set the icon, I do a pull request for this issue, would you mind if I changed the default icon for the LED?

    led.setIcon("mdi:led-outline"); // optional

image

JacobChrist avatar May 22 '23 15:05 JacobChrist

Oh that's a terrific addition, hell yes, thank you.

daniloc avatar May 22 '23 16:05 daniloc