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 Interlogix Ultrasync Alarm System Integration #15482

Closed
wants to merge 0 commits into from

Conversation

caronc
Copy link
Contributor

@caronc caronc commented Oct 30, 2020

Proposed change

Added documentation for new integration: Interlogix UltraSync Hub

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase: core/42549
  • Link to parent pull request in the Brands repository: brands/1994
  • This PR fixes or closes issue: n/a

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@probot-home-assistant probot-home-assistant bot added has-parent This PR has a parent PR in a other repo in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Oct 30, 2020
@probot-home-assistant probot-home-assistant bot added current This PR goes into the current branch Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! labels Oct 30, 2020
@caronc caronc changed the title Add Informix Ultrasync Alarm System Integration Add Interlogix Ultrasync Alarm System Integration Oct 30, 2020
@klaasnicolaas
Copy link
Member

Please rebase this pull request to the next branch

@caronc caronc changed the base branch from current to next October 30, 2020 02:20
@caronc caronc force-pushed the current branch 2 times, most recently from 9d367cd to 35dcdb3 Compare October 30, 2020 02:30
@caronc
Copy link
Contributor Author

caronc commented Oct 30, 2020

@klaasnicolaas Thanks for your feedback; done! 👍

@klaasnicolaas klaasnicolaas added next This PR goes into the next branch new-integration This PR adds documentation for a new Home Assistant integration and removed current This PR goes into the current branch needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Oct 30, 2020
description: Instructions on how to integrate your Interlogix UltraSync Hub within Home Assistant.
ha_category:
- Alarm
logo: informix.png
Copy link
Member

Choose a reason for hiding this comment

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

Please add a logo to our brands repository, you can remove the line here

Copy link
Contributor Author

@caronc caronc Oct 30, 2020

Choose a reason for hiding this comment

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

Done; updated comments. Here is a quick link to brands/1994
You also brought forth another bug, it shouldn't have read informix.png, but interlogix.png instead. Sorry about that; this is also fixed in latest push to this branch.

Copy link
Member

Choose a reason for hiding this comment

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

You'll need to remove the logo manifest from this PR.


Go to the integrations page in your configuration and click on new integration -> UltraSync.

**Note**: You can only be logged into the ZeroWire/UltraSync hub with the same user once; a subsequent login with the same user logs out the other. Since Home Assistant (HA) actively polls and maintains a login session to this Hub, it can prevent you from being able to log into at the same time elsewhere (via it's website). It is strongly recommended that you create a second user account on your Hub dedicated for just HA.
Copy link
Member

Choose a reason for hiding this comment

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

Don't abbreviate HA.

ha_domain: ultrasync
---

The `ultrasync` platform will allow you to monitor and control your Interlogix UltraSync Hub from within Home Assistant and setup automation.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The `ultrasync` platform will allow you to monitor and control your Interlogix UltraSync Hub from within Home Assistant and setup automation.
The Ultrasync integration will allow you to monitor and control your Interlogix UltraSync Hub from within Home Assistant and setup automation.


## Sensor

This component will create these sensors in the format of `{ultrasync_hubname}_{sensor}`; The below example assumes you accept the default name of `UltraSync` (which is still represented in lowercase):
Copy link
Member

Choose a reason for hiding this comment

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

The names entity names can be changed by the user, right? They are also visible in the device overview, right?

So the device entity name hint should not be needed.


## Sensor

This component will create these sensors in the format of `{ultrasync_hubname}_{sensor}`; The below example assumes you accept the default name of `UltraSync` (which is still represented in lowercase):
Copy link
Member

Choose a reason for hiding this comment

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

Additionally, we don't use double spacing.

- `ultrasync_area3state`: The Area 3 State
- `ultrasync_area4state`: The Area 4 State

There are several states each sensor can be at, but usually they will be one of the following: `Unknown`, `Ready`, `Not Ready`, `Armed Stay`, and `Armed Away`. The `Unknown` state is assigned to sensors that are not reporting; they usually sit in the spots of the Area's you're not monitoring.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
There are several states each sensor can be at, but usually they will be one of the following: `Unknown`, `Ready`, `Not Ready`, `Armed Stay`, and `Armed Away`. The `Unknown` state is assigned to sensors that are not reporting; they usually sit in the spots of the Area's you're not monitoring.
There are several states each sensor can be at, but usually, they will be one of the following: `Unknown`, `Ready`, `Not Ready`, `Armed Stay`, and `Armed Away`. The `Unknown` state is assigned to sensors that are not reporting; they usually sit in the spots of the areas you're not monitoring.


## Event Automation

When an Zone or Sensor changes it's state an event usable for automation is triggered on the Home Assistant Bus.
Copy link
Member

Choose a reason for hiding this comment

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

Suggestion: add support for device automations to the integration?

Suggested change
When an Zone or Sensor changes it's state an event usable for automation is triggered on the Home Assistant Bus.
When a zone or sensor changes its state an event is triggered. Those events are usable for automations.

- `ultrasync_sensor_update`: The event includes the sensor no, name, and new status it changed to.
- `ultrasync_area_update`: The event includes the area no, name, and new status it changed to (if it did). **Note**: Area's are sent periodic heartbeats in which case the state will not change at all.

Example automation to send a message via [Apprise](https://www.home-assistant.io/integrations/apprise/) on a sensor change in your home:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Example automation to send a message via [Apprise](https://www.home-assistant.io/integrations/apprise/) on a sensor change in your home:
Example automation to send a message via [Apprise](/integrations/apprise/) on a sensor change in your home:

Comment on lines 61 to 65
Available services:

- `stay`: Set alarm scene to Stay Mode
- `away`: Set alarm scene to Away Mode (fully activate Alarm)
- `disarm`: Disarm the alarm
Copy link
Member

Choose a reason for hiding this comment

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

Aren't these services dictated by the alarm control panel entity component?

@caronc caronc closed this Nov 9, 2020
@probot-home-assistant probot-home-assistant bot removed the in-progress This PR/Issue is currently being worked on label Nov 9, 2020
@probot-home-assistant probot-home-assistant bot added Invalid and removed Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! labels Nov 9, 2020
@caronc
Copy link
Contributor Author

caronc commented Nov 9, 2020

I pushed a new change and that some how triggered a close on this branch 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted has-parent This PR has a parent PR in a other repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants