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
Add Tuya Smart Water Timer #95053
Add Tuya Smart Water Timer #95053
Conversation
Hey there @tuya, @zlinoliver, @frenck, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
@frenck Any change we could get this reviewed, this is really simple modification. |
This PR has been open for just 7 hours (5 hours since your last comment), and you think it takes too long and it is time ping people? Please... don't do that. See also: https://developers.home-assistant.io/docs/review-process#what-not-to-do |
Why do you keep merging dev into this PR? Was there a conflict? An upstream change the affected this PR? It is good to be a little up-to-date, but you really don't have to merge constantly. ../Frenck |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Please do demand and ping for reviews. The queue is large. Please see our developer documentation for information on our review process (and on what not to do...). ../Frenck |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Hopefully this can now be merged :-) |
@frenck This is the diagnostic dump hope we can get this integrated :-) Thanks! |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Thanks |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as outdated.
This comment was marked as outdated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @syepes 👍
../Frenck
Breaking change
None
Proposed change
Adds support for Smart Water Timers from Tuya
This an updated version of #69188 and adds only the working elements
R4238 Smart Irrigation Control
https://wooxhome.com/products-c10/other-c4/woox-r4238-smart-garden-irrigation-control-p107
TUYA Integration: Added support for Moes Smart Water Timer
https://user-images.githubusercontent.com/795322/135816004-3268df58-b7e4-48d2-bbf7-cb40f481b234.jpg
https://www.moeshouse.com/products/smart-bluetooth-sprinkler-water-timer-with-rain-delay-filter-washer-programmable-irrigation-timer-with-automatic-and-manual-watering-for-standard-outdoor-garden-faucet-lawn-bluetooth-hub-required-for-irrigation-history
Standard Status Set
Type of change
Additional information
None
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: