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

feat(config): add Zooz Zen37 800LR Wall Remote #6577

Merged
merged 5 commits into from
Apr 10, 2024

Conversation

zackbcom
Copy link
Contributor

@zackbcom zackbcom commented Dec 29, 2023

Changes

Questions

I'm unsure if I need the following snippit of code. Seems like other Zooz devices have this.

	"compat": {
		// The device sends Configuration CC info reports in 4-byte chunks, causing each query to block the network for roughly 1.5 seconds.
		"skipConfigurationNameQuery": true,
		"skipConfigurationInfoQuery": true
	},

Testing artifacts

  • Tested on local UI. Everything looks good.
    • image
    • image

@zwave-js-assistant zwave-js-assistant bot added the config ⚙ Configuration issues or updates label Dec 29, 2023
@zackbcom zackbcom changed the title Draft: feat(config): add zen37 Draft: feat(config): add Zooz 800 Series Z-Wave Long Range Wall Remote ZEN37 800LR (Battery Powered) Dec 29, 2023
@zacs
Copy link
Contributor

zacs commented Feb 5, 2024

@AlCalzone anything I can do to help push this along?

@zackbcom
Copy link
Contributor Author

zackbcom commented Feb 6, 2024

@zacs I have been sick. If you want to finish the MR, feel free.

@zackbcom zackbcom changed the title Draft: feat(config): add Zooz 800 Series Z-Wave Long Range Wall Remote ZEN37 800LR (Battery Powered) feat(config): add Zooz 800 Series Z-Wave Long Range Wall Remote ZEN37 800LR (Battery Powered) Feb 10, 2024
@zackbcom zackbcom changed the title feat(config): add Zooz 800 Series Z-Wave Long Range Wall Remote ZEN37 800LR (Battery Powered) feat(config): add Zooz - Wall Remote - Zen37 800LR Feb 10, 2024
@zackbcom
Copy link
Contributor Author

@zacs @AlCalzone
Ready for review.

@eric10k93
Copy link

any chance of getting this reviewed/merged soon?

@l3arcf
Copy link
Contributor

l3arcf commented Mar 17, 2024

I too would love to see this reviewed and merged. Up-vote here.

@drewmcdowell
Copy link

Waiting with bated breath for this to be merged. 🤞

@justinsanker
Copy link

Hey guys, any chance you could complete this before the end of the month?

@AlCalzone
Copy link
Member

Did a few cleanup changes, otherwise looks good.
I'm awaiting confirmation from Zooz about the association groups - they might not be necessary.

@AlCalzone AlCalzone closed this Apr 10, 2024
@AlCalzone AlCalzone reopened this Apr 10, 2024
@AlCalzone AlCalzone changed the title feat(config): add Zooz - Wall Remote - Zen37 800LR feat(config): add Zooz Zen37 800LR Wall Remote Apr 10, 2024
@AlCalzone AlCalzone merged commit 11645ce into zwave-js:master Apr 10, 2024
22 of 29 checks passed
AlCalzone added a commit that referenced this pull request Apr 10, 2024
### Bugfixes
* Disallow associating a node with itself and skip self-associations when rebuilding routes (#6749)
* Fix computation of SNR margin when noise floor measurement is N/A (#6732)

### Config file changes
* Add new Leviton 800 series devices (#6757)
* Add UltraPro Z-Wave Plus In-Wall Toggle Switch, 700S (#6664)
* Rename generic 700 series controller to include 800 series (#6744)
* Add fingerprint and config parameters for UltraPro 700 Switch (#6726)
* Add Zooz Zen37 800LR Wall Remote (#6577)

### Changes under the hood
* Several dependency updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙ Configuration issues or updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing device configuration: Zooz ZEN37
7 participants