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 Bosch Smoke Alarm II (#9) #7354

Merged
merged 4 commits into from Nov 28, 2023
Merged

Conversation

Finne75
Copy link
Contributor

@Finne75 Finne75 commented Nov 13, 2023

  • Create bsd-2-smoke-alarm.js

  • Rename bsd-2-smoke-alarm.js to bsd-2-smoke-alarm.json

* Create bsd-2-smoke-alarm.js

* Rename bsd-2-smoke-alarm.js to bsd-2-smoke-alarm.json
@Finne75
Copy link
Contributor Author

Finne75 commented Nov 13, 2023

Hu? Validation error for DDF code provided by the deCONZ GUI DDF editor?

devices/bosch/bsd-2-smoke-alarm.json Outdated Show resolved Hide resolved
devices/bosch/bsd-2-smoke-alarm.json Outdated Show resolved Hide resolved
@SwoopX
Copy link
Collaborator

SwoopX commented Nov 13, 2023

Really good DDF. One question though: does the device respond to arbitrary read requests without waking it up?

@SwoopX SwoopX added this to the v2.24.2-beta milestone Nov 13, 2023
@Finne75
Copy link
Contributor Author

Finne75 commented Nov 14, 2023

Really good DDF. One question though: does the device respond to arbitrary read requests without waking it up?

The device responds to read requests, e.g. to read a cluster via GUI. It does so at any time as far as I noticed.

@manup
Copy link
Member

manup commented Nov 16, 2023

@SwoopX PR looks good to me, ready to merge?

@SwoopX
Copy link
Collaborator

SwoopX commented Nov 16, 2023

@manup Yeah, all looking good. You could remove the "awake": true parts, as the device seems to be a light sleeper, but that's just a blemish 😉

@manup manup merged commit a45506b into dresden-elektronik:master Nov 28, 2023
1 check failed
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.

None yet

3 participants