search

CJNE / ha-porscheconnect

Porsche Connect custom component for Home Assistant
MIT License
29 stars 12 forks source link

Lock entity #48

Closed fredriklj closed 2 years ago

fredriklj commented 2 years ago

First stab at a lock entity, is seemingly working. PIN must be given as "code" parameter to the service lock.unlock. Could use some help with unit testing :-)

Maybe I should have placed this against the dev branch, but discovered it too late! Should I move there?

CJNE commented 2 years ago

Great, thanks! I'll just add the missing tests for 100% coverage, nice job!

fredriklj commented 2 years ago

In all excitement I evidently lost track of which branch I was in when doing the changes :)