feat: add ability to set user pin codes and fix bug with fetching pin codes #66
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
As part of managing a vacation rental I wanted the ability to programmatically set august lock pin codes. Thanks to the APIs documented here https://github.com/jmaxxz/keymaker/blob/master/postman/august.postman_collection.json I was able to implement this.
First we need to perform an update pin operation and then we need to sync the pins to the lock.
Changes
Add new api method (
set_pin_for_user
) which performs both the update and the syncFix a bug in pin.py I was hitting when initially calling
get_pins()
due to a missing key. To do this we switch from direct dictionary access to using.get()