Skip to content
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 doc for infer_arming_state option in ness_alarm #9031

Conversation

nickw444
Copy link
Contributor

Description:
Introduce a new configuration option infer_arming_state which is passed down into the nessclient client library which attempts to fix an issue where the arming state is incorrectly reported shortly after the panel has been armed.

Pull request in home-assistant (if applicable): home-assistant/core#22379

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follows the standards.

@ghost ghost added the to-do label Mar 25, 2019
@klaasnicolaas klaasnicolaas added new-feature This PR adds documentation for a new Home Assistant feature to an existing integration ready-for-review This PR needs to be reviewed next This PR goes into the next branch has-parent This PR has a parent PR in a other repo and removed to-do labels Mar 25, 2019
@ghost ghost assigned klaasnicolaas Mar 25, 2019
@klaasnicolaas klaasnicolaas removed their assignment Mar 25, 2019
Copy link
Member

@klaasnicolaas klaasnicolaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Can be merged after merging the parent PR 👍

@klaasnicolaas klaasnicolaas added awaits-parent Awaits the merge of an parent PR parent-merged The parent PR has been merged already and removed ready-for-review This PR needs to be reviewed awaits-parent Awaits the merge of an parent PR labels Mar 25, 2019
@klaasnicolaas
Copy link
Member

Thnx for the PR @nickw444 👍

@klaasnicolaas klaasnicolaas merged commit 2da857c into home-assistant:next Mar 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-parent This PR has a parent PR in a other repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch parent-merged The parent PR has been merged already
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants