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

Convert Sonoff SNZB-02D & SNZB-02P to modernExtend #7572

Merged
merged 8 commits into from
May 30, 2024

Conversation

burmistrzak
Copy link
Contributor

Shouldn't have any adverse effects. Hopefully. 🤞

@burmistrzak
Copy link
Contributor Author

I'll have to re-pair one of each anyways, so I can see if reporting is actually required or not. 🤔

Related: Koenkk/zigbee2mqtt#5562 Koenkk/zigbee2mqtt#8971

src/devices/sonoff.ts Outdated Show resolved Hide resolved
src/devices/sonoff.ts Outdated Show resolved Hide resolved
@Koenkk Koenkk merged commit 825a7fa into Koenkk:master May 30, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented May 30, 2024

Thanks!

@Pferdebockwurst
Copy link
Contributor

Do I need to re-pair or reconfigure these after updating?

@burmistrzak
Copy link
Contributor Author

Do I need to re-pair or reconfigure these after updating?

@Pferdebockwurst I personally did just a reconfigure to ensure genPollCtrl is bound.
You'll have have to press the Pairing button on these devices (preferably once while you issue the command) to keep them awake.

@Pferdebockwurst
Copy link
Contributor

genPollCtrl

Ok thank you, I'll do this too and report back, if there are any problems.

@sKinyo
Copy link

sKinyo commented Jun 1, 2024

Was there a need to remove the device specific reporting settings?

Temperature
from
{min: 30, max: constants.repInterval.MINUTES_5, change: 20}
to generic
{min: '10_SECONDS', max: '1_HOUR', change: 100}

@burmistrzak
Copy link
Contributor Author

Was there a need to remove the device specific reporting settings?

Temperature from {min: 30, max: constants.repInterval.MINUTES_5, change: 20} to generic {min: '10_SECONDS', max: '1_HOUR', change: 100}

@sKinyo It's the default for modernExtend. Should also make the network a bit less busy.
Feel free to change it, in case you need more frequent updates. 😉

@satter
Copy link

satter commented Jun 2, 2024

My SNZB-02P sensor is missing OTA capabilities after the z2m upgrade: Koenkk/zigbee2mqtt#22889

This PR seems to be removing this capability from the device. Is this an intended removal, and is the device not capable of OTA updates?

@burmistrzak
Copy link
Contributor Author

My SNZB-02P sensor is missing OTA capabilities after the z2m upgrade: Koenkk/zigbee2mqtt#22889

This PR seems to be removing this capability from the device. Is this an intended removal, and is the device not capable of OTA updates?

AFAIK, it does support OTA, but we currently don't have any firmware images for it... 🫤
Do you have an OTA image for it? If so, please open a PR here: https://github.com/Koenkk/zigbee-OTA

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.

5 participants