FutureTense / lock-manager

Home Assistant Lock Manager
55 stars 19 forks source link

Do case insensitive comparison in lock_manager.yaml #54

Closed mew1033 closed 3 years ago

mew1033 commented 3 years ago

unknown != Unknown

firstof9 commented 3 years ago

@FutureTense if you want to, you can cherry pick this commit: https://github.com/firstof9/lock-manager/commit/3a9e57902d69bc94d0218a1c040d14517e2d35b0 to fix this

mew1033 commented 3 years ago

This PR makes the comparison work with both cases, but as long as its consistent, it shouldn't matter too much.

firstof9 commented 3 years ago

Ya I was waiting for some testing, but I'll have a PR to change the case, that's @mew1033 for catching this tho.