node-zwave-js
node-zwave-js copied to clipboard
feat(config): add new GDZW7-ECO - Ecolink 700 Series Garage Door Controller
Fixes #6570
Changes
- Added config for GDZW7-ECO - Ecolink 700 Series Garage Door Controller
- Params from debug gdzw7-eco_debug.json and product manual.
- Note device firmware had a different set of default Values compared the the manual. I have set the defaults to the ones from device firmware.
Testing artifacts
- I have tested this on the UI.
-
@AlCalzone What is the best way to test configs on the ui?
I'm running zwave-js-ui on a docker container. Am I able to copy the configs into the container and re-integrate the device to test these changes? Do you know the location if that is the easiest way?
Thanks.
Edit: I put them into my volume mount: /store/.config-db/devices/...
and got them to work.
@AlCalzone ready for review
Sorry for the delay - I just got back from parental leave. I'd have fixed this myself, but the comments about the defaults left me confused... What are the default values the device has when freshly reset and paired? The ones in the JSON file or the ones in the comments behind them?
Other than that I have a few minor remarks:
I have updated the default values to the one from the device firmware. The others were from the manual, which i have removed.
@AlCalzone Ready for review