Skip to content

add document for fan template#5296

Merged
balloob merged 1 commit into
home-assistant:nextfrom
giangvo:issue/add-template-fans
May 3, 2018
Merged

add document for fan template#5296
balloob merged 1 commit into
home-assistant:nextfrom
giangvo:issue/add-template-fans

Conversation

@giangvo
Copy link
Copy Markdown
Contributor

@giangvo giangvo commented May 2, 2018

Description:
Add document for template fans.

**Pull request in home-assistant

Checklist:

  • Branch: Fixes, changes and adjustments should be created against current. New documentation for platforms/components and features should go to next.
  • The documentation follow the standards.

@giangvo
Copy link
Copy Markdown
Contributor Author

giangvo commented May 2, 2018

@frenck @balloob please review.
Thanks

@balloob balloob merged commit fd5f1fc into home-assistant:next May 3, 2018
Danielhiversen added a commit that referenced this pull request May 3, 2018
@balloob
Copy link
Copy Markdown
Member

balloob commented May 6, 2018

This PR broke the build, I reverted the docs.

@balloob
Copy link
Copy Markdown
Member

balloob commented May 6, 2018

Please fix it and submit again

@balloob
Copy link
Copy Markdown
Member

balloob commented May 6, 2018

Found it, missed a " in the yaml e9aad88

@balloob
Copy link
Copy Markdown
Member

balloob commented May 6, 2018

Please in the future test your branches 🤔

@balloob
Copy link
Copy Markdown
Member

balloob commented May 6, 2018

And another missing quotes too that caused invalid yaml 3ca0be1

@giangvo
Copy link
Copy Markdown
Contributor Author

giangvo commented May 6, 2018

@balloob, I am sorry about that. I though we have branch build. Noted and will check my branch next time. Have u fixed the template?

@balloob
Copy link
Copy Markdown
Member

balloob commented May 7, 2018

It's fixed now 👍 . Somehow the test on this branch passed but then failed once they hit the main repo. That should have caught it normally.

@giangvo
Copy link
Copy Markdown
Contributor Author

giangvo commented May 7, 2018

👍Thank you!

@jeradM
Copy link
Copy Markdown
Member

jeradM commented May 7, 2018

@balloob I'm not seeing this fix in next and my local build is broken after rebasing. Should it be there already?

@balloob
Copy link
Copy Markdown
Member

balloob commented May 7, 2018

I put the fix in the rc branch, let me push that up to the next branch too

@jeradM
Copy link
Copy Markdown
Member

jeradM commented May 7, 2018

👍 thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants