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

Doorman L3 doorbell not available in 2023.09.X even when is should be #100297

Closed
anteuz opened this issue Sep 13, 2023 · 6 comments · Fixed by #100305
Closed

Doorman L3 doorbell not available in 2023.09.X even when is should be #100297

anteuz opened this issue Sep 13, 2023 · 6 comments · Fixed by #100305
Assignees

Comments

@anteuz
Copy link

anteuz commented Sep 13, 2023

The problem

The integration does not create a presence sensor for doorbell/doorbell events for the integrated doorbell is not available.

The Doorman L3 lock is "Finnish/Swedish" model, not really sure if they differ at all, but maybe it is a thing. I am more than happy to help with extracting logs / debugging where needed.

What version of Home Assistant Core has the issue?

core-2023.9.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

August / Yale

Link to integration documentation on our website

https://www.home-assistant.io/integrations/august/

Diagnostics information

config_entry-august-cde7bd1d7295ad6ad59e5d6a046192e2.json (2).txt

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @bdraco, mind taking a look at this issue as it has been labeled with an integration (august) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of august can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign august Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


august documentation
august source
(message by IssueLinks)

@bdraco
Copy link
Member

bdraco commented Sep 13, 2023

We only create them for type 7
https://github.com/bdraco/yalexs/blob/f473535b6f033078045ee1ac82a153530ed972ee/yalexs/lock.py#L113

It might be yours is a different type since its a different market.

If you enable debug logs for yalexs and reload the august integration you should be able to get the type in the logs

@anteuz
Copy link
Author

anteuz commented Sep 13, 2023

OK, that is the cause then, for my L3, the type is 10.

@anteuz
Copy link
Author

anteuz commented Sep 13, 2023

Debug log still catches the doorbell presses:
2023-09-13 16:56:10.849 DEBUG (MainThread) [yalexs.api_common] Processing activity: {'id': 'c0337e83-70fb-4c2a-8de1-58cf09b86a2a', 'timestamp': 1694602414000, 'icon': 'https://d3osa7xy9vsc0q.cloudfront.net/app/ActivityFeedIcons/[email protected]', 'action': 'lock_accessory_motion_detect', 'deviceID': '7DD595BB11C90345B8B27FC95095885E', 'deviceType': 'lock', 'title': 'Joku on lukon Etuovi luona sijainnissa Koti.'}

@anteuz
Copy link
Author

anteuz commented Sep 13, 2023

@bdraco
Copy link
Member

bdraco commented Sep 13, 2023

bdraco/yalexs#85

bdraco added a commit that referenced this issue Sep 13, 2023
changelog: bdraco/yalexs@v1.8.0...v1.9.0

fixes #100297

apparently the product ids are different for this device in the Finnish/Swedish market
@bdraco bdraco mentioned this issue Sep 13, 2023
20 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Oct 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants