Skip to content

Conversation

@burmistrzak
Copy link
Contributor

In #10309, the property key for the window detection switch was renamed from window_detection to window_open_mode which isn't recognized by the HA extension. This PR fixes the issue.

Depends on #10709

@Koenkk
Copy link
Owner

Koenkk commented Nov 12, 2025

To prevent another breaking change, I would suggest to add window_open_mode to https://github.com/Koenkk/zigbee2mqtt/blob/5797b56871215b8fac37cfeb4f702f73a09c4ae0/lib/extension/homeassistant.ts#L50

@burmistrzak
Copy link
Contributor Author

@Koenkk Sure, but a bit more has then to be changed as well.
https://github.com/Koenkk/zigbee2mqtt/blob/5797b56871215b8fac37cfeb4f702f73a09c4ae0/lib/extension/homeassistant.ts#L581

I can certainly do that if you prefer. However, reverting a key's name seemed like the less invasive option. 😊
Just let me know.

@Koenkk
Copy link
Owner

Koenkk commented Nov 16, 2025

hmm, then lets do the rename, could you update this pr to do just that?

@burmistrzak
Copy link
Contributor Author

@Koenkk Alright, should be good to go. 😊

@Koenkk Koenkk merged commit 786f58f into Koenkk:master Nov 16, 2025
3 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Nov 16, 2025

thanks!

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.

2 participants