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

Add doorbell suppression select #126

Closed
wants to merge 1 commit into from

Conversation

moeffju
Copy link

@moeffju moeffju commented Dec 31, 2022

This is a very quick and dirty, untested, first go at the code.

I don't have a full HA dev environment set up yet. I'll try to do that in the next days, but if there's someone else who wants to test this or correct obvious beginner's errors, have at it!

This is a very quick and dirty, untested first go at the code.
@kvj
Copy link
Owner

kvj commented Jan 5, 2023

Actually looks good to me! But I don't have the opener, so it would be nice if someone with the device could test the branch. You could remove the switches I've added earlier for that field

@moeffju
Copy link
Author

moeffju commented Jan 5, 2023

I'll try to get a test version deployed on my Home Assistant soon, I'm still recovering from The Covid rn.

@Zrnoile
Copy link

Zrnoile commented Jan 21, 2023

Can the option to press the bell on the opener be added? For example, to automate the camera's activation when someone presses the bell.

@Se7enair
Copy link

Se7enair commented Jan 30, 2023

Can the option to press the bell on the opener be added? For example, to automate the camera's activation when someone presses the bell.

This should be already possible. There is a "ring action". Binary_sensor that turns on when the bell is ringing. I already use this to open the camera on my dashboard.

@Se7enair
Copy link

I'll try to get a test version deployed on my Home Assistant soon, I'm still recovering from The Covid rn.

Actually looks good to me! But I don't have the opener, so it would be nice if someone with the device could test the branch. You could remove the switches I've added earlier for that field

I have a opener and tested it. I just copied the changed files to my homeassistant and restarted.

Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:328
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 11:29:55 (1 occurrences)
Last logged: 11:29:55

[redacted] Unable to serialize to JSON. Bad data found at $.event.a.select.nuki_opener_ring_suppression.a.options=dict_keys(['Off', 'Ring', 'Ring to Open', 'Continuous Mode', 'Ring & Ring to Open', 'Ring & Continuous Mode', 'Ring to Open & Continuous Mode', 'On (suppress all)'])(<class 'dict_keys'>
Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:254
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 11:30:41 (1 occurrences)
Last logged: 11:30:41

[redacted] Unable to serialize to JSON. Bad data found at $.result[234](State: select.nuki_opener_ring_suppression).attributes.options=dict_keys(['Off', 'Ring', 'Ring to Open', 'Continuous Mode', 'Ring & Ring to Open', 'Ring & Continuous Mode', 'Ring to Open & Continuous Mode', 'On (suppress all)'])(<class 'dict_keys'>

@moeffju
Copy link
Author

moeffju commented May 6, 2023

Superseded by #146 thanks @GNeras

@moeffju moeffju closed this May 6, 2023
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.

4 participants