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.

Sign up for GitHub

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

Jump to bottom

contact sensor state #18

Closed
keuninkske opened this issue Feb 28, 2021 · 1 comment · Fixed by #24
Closed

contact sensor state #18

keuninkske opened this issue Feb 28, 2021 · 1 comment · Fixed by #24

Comments

@keuninkske
Copy link

hello

i do get the error
[2/28/2021, 10:46:35 PM] [homebridge-volvo] This plugin generated a warning from the characteristic 'Contact Sensor State': characteristic value expected number and received undefined. See https://git.io/JtMGR for more info.

when I go to the mentioned link, it says i should contact the developper ...

i am in europe, have a volvo V60 and running homebridge v1.3.1 on raspberry pi
plugin: homebridge-volvo v1.0.5
drakkhen added a commit to drakkhen/homebridge-volvo that referenced this issue Sep 10, 2021
@drakkhen
Fix logic/syntax bugs with switch statements matching multiple values
b2ef622 
After a bit of tinkering, I figured out the cause of Issues trympet#18, trympet#22, and trympet#23.  The OR (||)
conditional for multiple case values was not working as intended -- they only matched the first
value in each case instead of any of the values separated by ||.

Converting those to use the fall-through feature of the switch statement resolved the problems.
@drakkhen drakkhen mentioned this issue Sep 10, 2021
Merged
@trympet trympet linked a pull request Sep 12, 2021 that will close this issue
Fix logic/syntax bugs with switch statements matching multiple values #24
Merged
@trympet
Copy link
Owner

trympet commented Sep 12, 2021

Fixed in #24

@trympet trympet closed this as completed Sep 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix logic/syntax bugs with switch statements matching multiple values drakkhen/homebridge-volvo
2 participants
@keuninkske @trympet