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

Enable OTA for some Develco devices #7561

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

samuli-as
Copy link
Contributor

@samuli-as samuli-as commented May 26, 2024

Enables the OTA attribute for some devices which now have images added in Koenkk/zigbee-OTA#502.
The humidity sensor and water leak sensor I have tested. I believe I picked the correct devices for the smart button and smart plug but I do not have them at hand to verify.

One thing I noticed after updating some devices that they were reporting every 30 seconds for some reason even though their reported values had not changed. This interval was not reflected in the Z2M reporting settings. Resetting them made them behave themselves. Any ideas where to look? I have some sensors left unpatched for further testing.

Edit: I wonder if this is related: Koenkk/zigbee2mqtt#21669

@mamrai1
Copy link
Contributor

mamrai1 commented May 26, 2024

If you are using ember adapter that is probably the case... I already replaced the batteries in both the develco heat alarms and changed the adapter to ezsp.

@samuli-as
Copy link
Contributor Author

Nope, I am on zstack.

@Koenkk
Copy link
Owner

Koenkk commented May 27, 2024

Can this be merged?

@samuli-as
Copy link
Contributor Author

I don't object to merging. The devices have been working fine otherwise but getting the reporting interval to work properly after updating (and not every 30 s no matter what) needed a reset. I don't know if that's something on our end or the firmware.

@samuli-as samuli-as marked this pull request as ready for review May 28, 2024 17:26
@Koenkk Koenkk merged commit 90d336e into Koenkk:master May 28, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented May 28, 2024

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.

3 participants