andrei-tatar / node-red-contrib-nora

Node Red Google Home integration
74 stars 25 forks source link

Lock Node #34

Closed rgerrans closed 4 years ago

rgerrans commented 4 years ago

Everything seems to be testing out right on my system.

A couple of quick notes:

rgerrans commented 4 years ago

@andrei-tatar I realized this never got pulled. Seems to be working fine on my deployment. Let me know any issues or concerns.

rgerrans commented 4 years ago

@andrei-tatar Let me know if I need to do anything else to this for you to pull (other than clean up the one conflict with the version number)?

andrei-tatar commented 4 years ago

@rgerrans did you manage to test it? I think it looks good. I can merge it once the conflict is resolved.

rgerrans commented 4 years ago

I've been using it for 2 1/2 months and seems to work fine. I fixed the version number in my branch. so should be good to go. I did want to flag my earlier comment:

  • I changed the configuration layout of the html file for the Lock node config. If you prefer a different structure, please feel free to edit. Or if you like this structure, I'm happy to apply it to the other nodes.
andrei-tatar commented 4 years ago

@rgerrans merged and published 0.0.33 Thanks for the contribution! :)

rgerrans commented 4 years ago

@andrei-tatar Happy to help. I need to get back on the fan work one of these days when life slows down a little....Thanks for all your work getting this up and maintaining it.