Skip to content

Add Legrand Home+ Control Integration document#16644

Merged
frenck merged 7 commits into
home-assistant:nextfrom
chemaaa:homepluscontrol
Mar 29, 2021
Merged

Add Legrand Home+ Control Integration document#16644
frenck merged 7 commits into
home-assistant:nextfrom
chemaaa:homepluscontrol

Conversation

@chemaaa
Copy link
Copy Markdown
Contributor

@chemaaa chemaaa commented Feb 19, 2021

Proposed change

This adds documentation for the Legrand Home+ Control integration that I am proposing to add to the Home Assistant core. This integration connects Home Assistant to the Legrand API ("Works with Legrand" platform) that can manage "Legrand/Bticino with Netatmo" products.

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.

@homeassistant
Copy link
Copy Markdown

Hi @chemaaa,

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!

@probot-home-assistant probot-home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Feb 19, 2021
@frenck frenck added the new-integration This PR adds documentation for a new Home Assistant integration label Feb 19, 2021
Copy link
Copy Markdown
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.

I think the name of the file should be homepluscontrol.markdown

Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Co-authored-by: Klaas Schoute <klaas_schoute@hotmail.com>
@chemaaa
Copy link
Copy Markdown
Contributor Author

chemaaa commented Feb 28, 2021

I think the name of the file should be homepluscontrol.markdown

Thanks. Yes, the naming is expected to change to home_plus_control based on the core component code review. I will be making the change to this documentation file as soon as I complete the code refactoring.

Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
Comment thread source/_integrations/legrand_homepluscontrol.markdown Outdated
@frenck frenck added the in-progress This PR/Issue is currently being worked on label Mar 1, 2021
@chemaaa
Copy link
Copy Markdown
Contributor Author

chemaaa commented Mar 6, 2021

Note that the component has been renamed home_plus_control (instead of homepluscontrol), so the markdown filename has been modified to reflect this as well.

@MartinHjelmare MartinHjelmare added the parent-merged The parent PR has been merged already label Mar 25, 2021
Comment thread source/_integrations/home_plus_control.markdown Outdated
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.

Thanks, @chemaaa 👍

@frenck frenck merged commit b304c0f into home-assistant:next Mar 29, 2021
@probot-home-assistant probot-home-assistant Bot removed in-progress This PR/Issue is currently being worked on parent-merged The parent PR has been merged already labels Mar 29, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 30, 2021
@chemaaa chemaaa deleted the homepluscontrol branch April 5, 2021 10:16
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 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.

5 participants