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

Add support for HomeKit Controller Locks #19867

Merged
merged 7 commits into from Jan 12, 2019

Conversation

Projects
None yet
3 participants
@adrum
Copy link
Contributor

adrum commented Jan 8, 2019

Description:

This adds support for HomeKit locks to appear in Home Assistant.

CC: @mjg59

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8097

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.

@MartinHjelmare MartinHjelmare changed the title Added support for HomeKit Controller Locks Add support for HomeKit Controller Locks Jan 9, 2019

adrum added some commits Jan 10, 2019

@MartinHjelmare
Copy link
Member

MartinHjelmare left a comment

Great!

@MartinHjelmare MartinHjelmare merged commit 7676b3f into home-assistant:dev Jan 12, 2019

5 checks passed

Hound No violations found. Woof!
WIP ready for review
Details
cla-bot Everyone involved has signed the CLA
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 93.038%
Details

@wafflebot wafflebot bot removed the in progress label Jan 12, 2019

@adrum adrum deleted the adrum:feature/homekit-controller/lock branch Jan 12, 2019

mxworm added a commit to mxworm/home-assistant that referenced this pull request Jan 12, 2019

Merge branch 'dev' into current
* dev: (150 commits)
  Update coveragerc file
  Upgrade greeneye_monitor to 1.0 (home-assistant#19631)
  Update doorbird events to include URLs on event_data (home-assistant#19262)
  Support for html5 notifications to suggest their names (home-assistant#19965)
  catch TypeError's in addition to ValueError's for unifi direct device tracker (home-assistant#19994)
  'latest_dir' referenced before assignment (home-assistant#19952)
  Repackage ZHA component (home-assistant#19989)
  Add support for HomeKit Controller Locks (home-assistant#19867)
  Don't set friendly_name in Zha entity. (home-assistant#19991)
  Lint
  Wink: Update pubnubsub-handler version to make it compatible with python 3.7 (home-assistant#19625)
  Upgrade pytest-cov to 2.6.1 (home-assistant#19988)
  Upgrade huawei-lte-api to 1.1.3 (home-assistant#19987)
  Support for multiple Fibaro gateways (home-assistant#19705)
  Split locative to a separate component (home-assistant#19964)
  Bumped version to 0.86.0.dev0
  Fix fail2ban tests
  Expose more information about shipments by PostNL (home-assistant#18334)
  Fix the anthemav component by removing a debugging line. (home-assistant#19979)
  Add support for 'via_hub' for device_info (home-assistant#19454)
  ...

# Conflicts:
#	homeassistant/components/homematicip_cloud/__init__.py
#	requirements_all.txt
#	requirements_test_all.txt

@balloob balloob referenced this pull request Jan 23, 2019

Merged

0.86.0 #20354

alandtse added a commit to alandtse/home-assistant that referenced this pull request Feb 12, 2019

Add support for HomeKit Controller Locks (home-assistant#19867)
* Added HomeKit Controller Lock

* cleaned up code according to standards

* fixed lint issues

* added private constant for jammed state

* removed state_unknown
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment