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

Enable setting alarm mode night for arlo platform #20143

Merged
merged 2 commits into from Jan 16, 2019

Conversation

Projects
None yet
4 participants
@keslerm
Copy link
Contributor

keslerm commented Jan 15, 2019

Description:

Small addition to the Arlo alarm control panel to be able to use the Night alarm mode configuration.

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:

@homeassistant

This comment has been minimized.

Copy link

homeassistant commented Jan 15, 2019

Hi @keslerm,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@frenck

This comment has been minimized.

Copy link
Member

frenck commented Jan 16, 2019

@keslerm The documentation PR is created on your fork, instead of on our repositories 😉
Adding the docs-missing label until this is resolved.

@frenck frenck added the docs-missing label Jan 16, 2019

@keslerm keslerm referenced this pull request Jan 16, 2019

Merged

Update alarm_control_panel.arlo.markdown #8191

2 of 2 tasks complete
@keslerm

This comment has been minimized.

Copy link
Contributor Author

keslerm commented Jan 16, 2019

@frenck whoops, fixed it!

@balloob balloob removed the docs-missing label Jan 16, 2019

@balloob balloob merged commit 29c6584 into home-assistant:dev Jan 16, 2019

5 checks passed

Hound No violations found. Woof!
WIP Legacy commit status override — see details
Details
cla-bot Everyone involved has signed the CLA
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.02%) to 92.958%
Details

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

@balloob balloob referenced this pull request Jan 23, 2019

Merged

0.86.0 #20354

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