-
-
Notifications
You must be signed in to change notification settings - Fork 623
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
feat(config): added 11 Shelly Qubino Wave devices #6633
feat(config): added 11 Shelly Qubino Wave devices #6633
Conversation
Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM. added wave_template
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM added wave_template
I'm sure you saw the failed tests. Most are because the "label" is above the "import". |
@apella12 Yes, I'm fixing that right now. Is there any way to re-run the tests without doing a new pull? Sorry for the question not so skilled in programming and git |
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM added wave_template fixed errors from previous pull request feat(config): added Wave devices #6610 Fixes reportd by the lint test
more fixes of the code
@QubinoHelp Is there any way to re-run the tests without doing a new pull? No that I'm aware of. (I'm assuming you mean after a new commit is added to the PR?) You don't have to delete and start over like you did, just keep adding commits to fix the issues. I don't know how you are doing this, but you should use VSC, it should show the errors as you go. |
I use the VSC I don't have set check all of the code, need more time to set that too :) |
Last effort looks close. I think it is just step 2 (zwave lint has passed) at the bottom on this posting to clean up the files. |
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM, 1 Mini, 1PM Mini, PM Mini added wave_template added zwaveAllianceId to Wave 1, Wave 1PM, Wave Plug US, Wave Plug UK, Wave i4, Wave Pro 1, Wave 1PM Mini, Wave 1 Mini change optimise Wave 1, Wave 1PM, Wave 2PM config using template change manufacturer name
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM, 1 Mini, 1PM Mini, PM Mini added wave_template added zwaveAllianceId to Wave 1, Wave 1PM, Wave Plug US, Wave Plug UK, Wave i4, Wave Pro 1, Wave 1PM Mini, Wave 1 Mini change optimise Wave 1, Wave 1PM, Wave 2PM config using template change manufacturer name
add Wave 2PM missing parameters add Wave Plug UK mising parameter fix Wave Pro 1PM fixed name
Hi, I tested the configuration for Wave I4. My device is well recognized, however I cannot make an association with a socket to turn it on/off, "Unable to add Node xx to Group xx of Node xx". Does it work for you? Tested with an association "On/Off [SW1]" in "Root Endpoint" for "Node Endpoint" and "No Endpoint" for "Target Endpoint". |
Hi @duylong Set it like this: Just done the test and it is working to me, pay attention that both of the Z-Wave devices are included with the same security level. |
I had integrated it in secure mode, thanks for the tip, it's perfect now. |
For information, the association of a Wave i4 in push button mode (momentary switch) with a Wave 1 configured in push button mode (momentary switch) (with an Auto-Off at 1 second) does not seem to work. I have to click several times to get my garage door to react. Key pressed send an BASIC_SET ON and key released send BASIC_SET OFF ? or Key pressed send an BASIC_SET ON and key pressed again send an BASIC_SET OFF ? Maybe my usage is not suited to the Wave i4 and I need to use a Wave 1 instead. |
Hi @duylong Did you open also a ticket on the Shelly support portal, since I get the same question there ;) If not, below the answer. The association is linked to the inputs and will only activate upon pressing the corresponding input button. Sending the command "ON" triggers Wave 1 to turn on and after a set amount of time, it will turn off. Although this is reported to the gateway, it is not reported back to the device that sent the command (i4), so it is unaware that Wave 1 is now off. The i4 believes the last command sent was "ON" and expects the next command to be "OFF". Therefore, pressing the i4 a second time will not affect Wave 1 as it will receive the "OFF" command, and the next press will send the "ON" command again. Luckily this have nothing to do with the configuration files I've made the pull request, anyhow thanks for letting us know. |
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! I cleaned up the templates a bit, removed some unnecessary fields and restored the QNSH-001P10 file which I had cleaned up when you added it.
### Bugfixes * Add workaround for devices that omit notification event parameters instead of sending "no data" (#6719) ### Config file changes * Added 11 Shelly Qubino Wave devices (#6633) * Add Heatit Leakage Water Stopper (#6605) * Add Ring Smoke/CO Listener (#6591) * Add ZVIDAR Z-TRV-V01 thermostatic valve (#6542) * Add Safe Grow NSG-AB-02 Z-Wave Plus Smart Outlet Plug (#6535) * Add a new productId and add parameters to 14297/ZW1002 outlet (#6517)
added Wave devices i4, i4DC, Plug UK, Plug US, PRO 1, PRO 2, PRO 2, PRO 2PM
added wave_template
fixed errors from previous pull request feat(config): added Wave devices #6610