Skip to content

New IHC platform#4129

Merged
frenck merged 7 commits into
home-assistant:nextfrom
dingusdk:next
Jan 22, 2018
Merged

New IHC platform#4129
frenck merged 7 commits into
home-assistant:nextfrom
dingusdk:next

Conversation

@dingusdk
Copy link
Copy Markdown
Contributor

@dingusdk dingusdk commented Dec 3, 2017

Description:

Documentation of the IHC platform

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

@fabaff fabaff added the new-integration This PR adds documentation for a new Home Assistant integration label Dec 3, 2017
footer: true
logo: ihc.png
ha_category: Binary Sensor
ha_release: 0.59
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0.62

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

ha_iot_class: "Local Push"
---

Before you can use the IHC Binary Sensor platform, you must setup the [IHC Component](../ihc/)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does the relative link work?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It does work because the url has an ending / but I have changed all links to absolute /components/... (I can see that is how it is done on the other pages)

...
```

Configuration variables:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated
footer: true
logo: ihc.png
ha_category: Hub
ha_release: "0.59"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0.62

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated
---

IHC Controller integration for Home Assistant allows you to connect the LK IHC controller to Home Assistant.
(The controller is sold under other names in different countries - "ELKO Living system" in Sweeden and Norway)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"Sweden"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/light.ihc.markdown Outdated
...
```

Configuration variables:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/sensor.ihc.markdown Outdated
footer: true
logo: ihc.png
ha_category: Sensor
ha_release: 0.59
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0.62

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/sensor.ihc.markdown Outdated
- id: 12346
...
```
Configuration variables:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/switch.ihc.markdown Outdated
footer: true
logo: ihc.png
ha_category: Switch
ha_release: 0.59
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

0.62

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/switch.ihc.markdown Outdated
....
```

Configuration variables:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated

### Auto setup of IHC products

If auto setup is enabled, the ihc component will automatically find IHC products and insert these as devices in Home Assistant.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • Place backticks around the ich when referencing to the component.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated

If auto setup is enabled, the ihc component will automatically find IHC products and insert these as devices in Home Assistant.
To disable this set auto_setup to False. (Auto setup is on by default)
See the individual device types for a list of IHC products to be regocnized automatically.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • regocnized -> recognized

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated
Each device is associated with an IHC resource id.
To manually setup components you specify resource ids from the IHC project.
(The IHC project is the file you edit/upload to the IHC Controller using LK IHC Visual - or similar program if your controller is not the LK brand).
The project file is a XML file and you can view it with any text/xml editor.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • a XML -> an XML
  • text/xml -> text/XML

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated
To manually setup components you specify resource ids from the IHC project.
(The IHC project is the file you edit/upload to the IHC Controller using LK IHC Visual - or similar program if your controller is not the LK brand).
The project file is a XML file and you can view it with any text/xml editor.
You can rename it to have the xml extension and use a browser like Chrome or Internet Explorer.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • xml -> XML

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated

- **auto_setup** (*Optional*): True to have IHC products auto setup.
- **host** (*Required*): The url of the IHC Controller.
- **info** (*Optional*): If True additional ihc info will be shown on each component.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • ich -> ICH

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

IHC - Done

Comment thread source/_components/ihc.markdown Outdated
Configuration variables:

- **auto_setup** (*Optional*): True to have IHC products auto setup.
- **host** (*Required*): The url of the IHC Controller.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  • url -> URL

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

@frenck
Copy link
Copy Markdown
Member

frenck commented Jan 20, 2018

@dingusdk You documentation PR has been reviewed by @MartinHjelmare and myself.
We both left some change requests. Could you please take a look? Awesome! 👍

@dingusdk
Copy link
Copy Markdown
Contributor Author

@MartinHjelmare and @frenck - I have been going through your comments, and all should have been fixed now.

required: true
type: int
inverting:
description: If True the sensor will be inverted. Default False.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You should be able to put the default value as a separate key value pair with key default.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/ihc.markdown Outdated
Components will get a default name that is a combination of the IHC group and IHC resource id.
If you want to change the display names use the [Customizing entities](/docs/configuration/customizing-devices/)

### {% linkable_title Manualy setup %}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"Manual setup"
Make sure links to anchor are adjusted.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

{% endconfiguration %}

The resource id should be an id of a boolean IHC resource.
For more information about IHC resource ids see [Manual Setup](/components/ihc/#manualy-setup)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Adjust anchor after fix below.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/light.ihc.markdown Outdated

In the example above 12345 is ihc resource id and "tablelight" is the name.
The IHC resource id can be a light level for dimmers or an boolean output of a relay.
For more information about IHC resource ids see [Manual Setup](/components/ihc/#manualy-setup)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Adjust anchor.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Comment thread source/_components/switch.ihc.markdown Outdated
{% endconfiguration %}

The resource id should be a boolean resource. (On/Off)
For more information about IHC resource ids see [Manual Setup](/components/ihc/#manualy-setup)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Adjust anchor.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

I agree as well 👍
Thanks @dingusdk 🏅

@frenck frenck merged commit 412869f into home-assistant:next Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-integration This PR adds documentation for a new Home Assistant integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants