Skip to content

Initial STIEBEL ELTRON documentation#8763

Merged
klaasnicolaas merged 10 commits into
home-assistant:nextfrom
fucm:stiebel_eltron
Apr 15, 2019
Merged

Initial STIEBEL ELTRON documentation#8763
klaasnicolaas merged 10 commits into
home-assistant:nextfrom
fucm:stiebel_eltron

Conversation

@fucm
Copy link
Copy Markdown
Contributor

@fucm fucm commented Feb 27, 2019

Description:
Add initial STIEBEL ELTRON documentation.

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

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 Feb 27, 2019
@frenck frenck added new-integration This PR adds documentation for a new Home Assistant 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 another repo and removed to-do labels Feb 27, 2019
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.

The config variables do not match those in the parent PR.

In the python code I see name as required and hub as optional. In the documentation you have both set as optional.

Comment thread source/_components/climate.stiebel_eltron.markdown
Comment thread source/_components/stiebel_eltron.markdown Outdated
@fucm
Copy link
Copy Markdown
Contributor Author

fucm commented Apr 13, 2019

home-assistant PR is ready to be merged. Can this be approved as well?

Comment thread source/_components/stiebel_eltron.markdown Outdated
MartinHjelmare
MartinHjelmare previously approved these changes Apr 14, 2019
@klaasnicolaas klaasnicolaas added the parent-merged The parent PR has been merged already label Apr 14, 2019
klaasnicolaas
klaasnicolaas previously approved these changes Apr 14, 2019
@klaasnicolaas klaasnicolaas removed their assignment Apr 15, 2019
@klaasnicolaas klaasnicolaas merged commit 89d16ab into home-assistant:next Apr 15, 2019
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Apr 15, 2019
@klaasnicolaas
Copy link
Copy Markdown
Member

Thnx for the PR! 👍

@fucm fucm deleted the stiebel_eltron branch April 17, 2019 20:05
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 another repo new-integration This PR adds documentation for a new Home Assistant 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.

4 participants