Skip to content
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

Added Temperature, Humidity & Contact Sensor Post/Call Back to SmartThings #207

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JZ-SmartThings
Copy link

I added Temperature, Humidity & Contact Sensor Post Back to SmartThings. Otherwise I had to use lame workarounds (like syncing with a switch). Modifying the SmartApp was so much simpler than any other solution. 10 lines of code but world of difference with these 3 capabilities and their read-only limitations in SmartThings.

Ultimately, being able to post these back to ST were big missing pieces for me. @stjohnjohnson I'd appreciate my enhancement pull becoming part of your release and respect your work very much! As an FYI, most of my code is a subset of pull #194 + a few more lines. However, the pull is just more narrow is scope and thus less code to verify. @gandazgul thanks for that pull.

@davidcreager
Copy link

Thanks for this, I needed it!!! I had to make a small change, not sure if it is a bug or not, but the code assumes the temperature is an int and at line 775 in the smartapp you have 'device.setTemperature(value as int), I simply replaced this with 'device.setTemperature(value)'

Thanks again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants