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

fix(inovelli): add missing param 121 FanTimerMode #7023

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

JefeDavis
Copy link
Contributor

fanTimerMode was implemented on herdsman in Koenkk/zigbee-herdsman#881

new params were introduced in #6936 but fanTimerMode was missed, this PR adds that missing parameter

@Koenkk
Copy link
Owner

Koenkk commented Feb 5, 2024

@InovelliUSA could you review this?

@JefeDavis
Copy link
Contributor Author

@Koenkk any chance we can get this change in before the release?
@InovelliUSA do you have a sec to review?

@blubaru14
Copy link
Contributor

@InovelliUSA just bumping this. Thanks

@InovelliUSA
Copy link
Contributor

Yes, this looks good to be merged.

@Koenkk Koenkk merged commit 0d71ea4 into Koenkk:master Mar 1, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Mar 1, 2024

Thanks!

lucasteligioridis added a commit to lucasteligioridis/zigbee-herdsman-converters that referenced this pull request Mar 1, 2024
…-converters

* 'master' of github.com:lucasteligioridis/zigbee-herdsman-converters:
  chore(master): release 18.42.0 (Koenkk#7140)
  fix(ignore): update dependencies (Koenkk#7141)
  feat: Support fanTimerMode for Inovelli VZM35-SN (Koenkk#7023)
  fix: Reference attributes directly from cluster for various Schneider Electric specific attributes (Koenkk#7139)
lucasteligioridis pushed a commit to lucasteligioridis/zigbee-herdsman-converters that referenced this pull request Mar 1, 2024
* fix(inovelli): add missing param 121 FanTimerMode

* Update inovelli.ts

---------

Co-authored-by: Koen Kanters <[email protected]>
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.

None yet

4 participants