-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature Request: Expose LCD On and Off? #5
Comments
Did you try the functions directly? I couldn’t get the backlight functions
to work, so I didn’t include it... but maybe it could be due to the
specific backpacks I have.
…On Sat, Jan 5, 2019 at 7:11 PM Scott C. Lemon ***@***.***> wrote:
When I looked at the i2clcd npm module, it has the ability to turn on and
off the LCD backlight.
I was curious if you might consider exposing this ability to node-red,
maybe adding a few new keywords like "on" and "off"?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#5>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AB1hfTHBgq5NkZlrGrQKie_BjD0l7msOks5vAT84gaJpZM4ZyLQ_>
.
|
I wrote the issue before looking at your code. It was too easy and straight forward ... and it works on my units! I simply inserted the following lines, after your "init" if statement: if (msg.topic.localeCompare("on") == 0) {
lcd.on();
}
if (msg.topic.localeCompare("off") == 0) {
lcd.off();
} It works perfectly in Node-RED! I send the on/off commands and my backlight turns on and off! |
Great! This was exactly what didn't work for me so I never added it - probably due to my hardware. Feel free to submit a pull request and I'll merge it in. Thanks! |
update: actually, looks like I was trying to use the setBacklight() function instead of just the on/off ones. the latter works fine. I'll push the updates. Thanks again! |
When I looked at the i2clcd npm module, it has the ability to turn on and off the LCD backlight.
I was curious if you might consider exposing this ability to node-red, maybe adding a few new keywords like "on" and "off"?
The text was updated successfully, but these errors were encountered: