Skip to content

Add Overkiz documentation for new integration#20841

Merged
bdraco merged 4 commits intohome-assistant:nextfrom
iMicknl:overkiz/base_and_sensor_docs
Dec 24, 2021
Merged

Add Overkiz documentation for new integration#20841
bdraco merged 4 commits intohome-assistant:nextfrom
iMicknl:overkiz/base_and_sensor_docs

Conversation

@iMicknl
Copy link
Copy Markdown
Member

@iMicknl iMicknl commented Dec 23, 2021

Proposed change

Adds the first version of the Overkiz documentation, for the sensor entity.

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

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 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 Dec 23, 2021
@probot-home-assistant probot-home-assistant Bot added the next This PR goes into the next branch label Dec 23, 2021
@probot-home-assistant
Copy link
Copy Markdown

It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the current branch. Please change the target branch of this PR to current and rebase if needed. If this is documentation for a new feature, please add a link to that PR in your description.
(message by DocsTargetBranch)

@probot-home-assistant probot-home-assistant Bot added the has-parent This PR has a parent PR in another repo label Dec 23, 2021
@frenck frenck added the parent-merged The parent PR has been merged already label Dec 23, 2021
Copy link
Copy Markdown
Member

@bdraco bdraco left a comment

Choose a reason for hiding this comment

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

The somfy changes should probably be in a separate PR after this is merged since this PR is really about overkiz and not somfy

@bdraco
Copy link
Copy Markdown
Member

bdraco commented Dec 24, 2021

@iMicknl For the rest of these, please do one docs PR to match up with the core PR so they are easier to keep track of. Thanks 👍

@bdraco bdraco merged commit d26c8ed into home-assistant:next Dec 24, 2021
@probot-home-assistant probot-home-assistant Bot removed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch in-progress This PR/Issue is currently being worked on parent-merged The parent PR has been merged already labels Dec 24, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 25, 2021
@iMicknl iMicknl deleted the overkiz/base_and_sensor_docs branch December 27, 2021 09:55
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.

@iMicknl Please address these review comments in a new PR.

Thanks 👍

- Button
- Lock
- Number
ha_release: 2022.1.0
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.

This version is incorrect. It should tag the major version and above all 2022.1 won't be released.

Please open a PR to correct this to 2022.2


**Execution queue is full on gateway: #xxxx-xxxx-xxxx (soft limit: 10)**

The Overkiz has a limit of 10 actions that can be executed in parallel. Currently this can only be solved by adding a time-out in between your executions, where you need to take the execution time into account.
Copy link
Copy Markdown
Member

@frenck frenck Dec 27, 2021

Choose a reason for hiding this comment

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

This sounds like a bug to me, and something that should not be solved in the documentation.

We should not have to burden the end-user with these kinds of things.

Copy link
Copy Markdown
Member

@bdraco bdraco Dec 27, 2021

Choose a reason for hiding this comment

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

@iMicknl AFAICT this is a limitation on Somfy's side with no clear solution. Feel free to reach out on discord to discuss ideas to work around this implementation flaw (on Somfy's side).

- number
---

The Overkiz (by Somfy) integration platform is used by many different vendors, like Somfy, Hitachi and Atlantic. This integration will allow users to integrate their devices into Home Assistant using the Overkiz API.
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.

  • like Somfy, Hitachi and Atlantic -> like Somfy, Hitachi, and Atlantic


**Server busy, please try again later. (Too many executions)**

During peak hours, it could happen that the Overkiz platform is unable to execute your command. The integration will try to retry this command, however this is not guaranteed to succeed.
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.

  • Space after succeed.
  • Missing comma after however: this command, however, this is


### Z-Wave, Hue and Sonos not supported

Even though most Overkiz hubs supports adding Z-Wave, Hue and Sonos devices, this isn't supported in the Overkiz integration. All these platforms have native integrations in Home Assistant which are more stable and feature rich.
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.

  • supports -> support
  • Z-Wave, Hue and Sonos devices -> Z-Wave, Hue, and Sonos devices
  • feature rich -> feature-rich`


### Internet connectivity required

This integration communicates via the cloud based Overkiz API, since Overkiz doesn't expose a local API on their hubs. If you are only using Somfy IO compatible devices, you could purchase a Velux KLF200 hub and use to [the Velux integration](/integrations/velux/) which has a local API.
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.

  • cloud based -> cloud-based
  • Misplaced to, remove it: Velux KLF200 hub and use to -> Velux KLF200 hub and use

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

Labels

cla-signed has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants