Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added vera scene documentation #3996

Merged
merged 6 commits into from
Dec 9, 2017
Merged

Conversation

alanfischer
Copy link
Contributor

@alanfischer alanfischer commented Nov 16, 2017

Description:
This adds documentation for vera scenes

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

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.

@frenck frenck added the new-integration This PR adds documentation for a new Home Assistant integration label Nov 19, 2017
Copy link
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 for the PR @alanfischer. I've reviewed it and left some comments. Could you please take a look? Awesome!

layout: page
title: "Vera Scene"
description: "Instructions how to integrate Vera Scenes into Home Assistant."
date: 2015-10-20 21:00
Copy link
Member

Choose a reason for hiding this comment

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

Incorrect date?

logo: vera.png
ha_category: Scene
ha_iot_class: "Local Push"
ha_release: pre 0.7
Copy link
Member

Choose a reason for hiding this comment

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

The version seems unlikely... 😉

---
layout: page
title: "Vera Scene"
description: "Instructions how to integrate Vera Scenes into Home Assistant."
Copy link
Member

Choose a reason for hiding this comment

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

Typo: Instructions on how to

frenck
frenck previously approved these changes Nov 20, 2017
@frenck
Copy link
Member

frenck commented Nov 20, 2017

Thank you for updating the PR @alanfischer 🏅
Can be merged as soon as the parent PR gets merged.

logo: vera.png
ha_category: Scene
ha_iot_class: "Local Push"
ha_release: 0.59.0
Copy link
Member

Choose a reason for hiding this comment

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

"0.60"

@alanfischer
Copy link
Contributor Author

The ha_release version is now updated.

logo: vera.png
ha_category: Scene
ha_iot_class: "Local Push"
ha_release: 0.60.0
Copy link
Member

Choose a reason for hiding this comment

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

It should be "0.60", with quotes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, thanks!

MartinHjelmare
MartinHjelmare previously approved these changes Dec 7, 2017
Copy link
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.

Thanks!

@frenck frenck added this to the 0.60 milestone Dec 9, 2017
@frenck frenck self-assigned this Dec 9, 2017
@frenck frenck merged commit 7cf60f0 into home-assistant:next Dec 9, 2017
tschmidty69 pushed a commit to tschmidty69/home-assistant.github.io that referenced this pull request Dec 9, 2017
* Added vera scene documentation

* ✏️ Tiny grammar fix

* Updated vera scene documentation with proper description, date, and release

* Fixed ha_release

* Fixed ha_release

* ✏️ Minor tweak
tschmidty69 pushed a commit to tschmidty69/home-assistant.github.io that referenced this pull request Dec 10, 2017
* Added vera scene documentation

* ✏️ Tiny grammar fix

* Updated vera scene documentation with proper description, date, and release

* Fixed ha_release

* Fixed ha_release

* ✏️ Minor tweak
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.

3 participants