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

DDF for SONOFF motion sensor SNZB-03P #7830

Merged
merged 4 commits into from
Aug 7, 2024

Conversation

Smanar
Copy link
Collaborator

@Smanar Smanar commented Jun 30, 2024

By @febalci
Product name : SONOFF SNZB-03P Motion Sensor Link
Manufacturer : SONOFF
Model identifier : SNZB-03P
Device type to add : Sensor

See #7821

@Smanar Smanar linked an issue Jun 30, 2024 that may be closed by this pull request
1 task
@Smanar Smanar mentioned this pull request Jun 30, 2024
1 task
Copy link
Contributor

github-actions bot commented Jun 30, 2024

Hey @Smanar, thanks for your pull request!

Tip

Modified bundles can be downloaded here.
Relative expire date

DDB changes

Modified

  • sonoff/snzb-03p_motion_sensor.json : Motion sensor ✔️

Validation

Tip

Everything is fine !

🕟 Updated for commit 6c354a1

Copy link
Collaborator

@SwoopX SwoopX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few quick things to check/amend, nothing serious 👍

devices/sonoff/snzb-03p_motion_sensor.json Outdated Show resolved Hide resolved
},
{
"name": "config/delay",
"refresh.interval": 3600,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to doublecheck the refresh interval, as reporting is significantly lower

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In a perfect world, "refresh.interval" is never used, it's just a security for me, the setting is only changed 2 times in a year.
Have just copy/paste other DDF (working I hope)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, absolutely agree. I guess it doesn't matter anyway if 1-2 reports get lost, the poll is very unlikely to happen at all.

devices/sonoff/snzb-03p_motion_sensor.json Outdated Show resolved Hide resolved
devices/sonoff/snzb-03p_motion_sensor.json Outdated Show resolved Hide resolved
devices/sonoff/snzb-03p_motion_sensor.json Outdated Show resolved Hide resolved
@manup manup added this to the v2.28.0-beta milestone Jul 10, 2024
@manup manup changed the title DDF add support for SONOFF SNZB-03P DDF for SONOFF motion sensor SNZB-03P Aug 7, 2024
@manup manup merged commit afd5261 into dresden-elektronik:master Aug 7, 2024
1 check passed
Copy link
Contributor

github-actions bot commented Aug 7, 2024

This pull request is now merged. The new DDB files have been uploaded to the store.

DDB Files

Modified

  • sonoff/snzb-03p_motion_sensor.json : Motion sensor : with hash (3a151d21cb)

🕖 Updated for commit afd5261

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SONOFF SNZB-03P
3 participants