Skip to content

Add juicenet platform #2670

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

Merged
merged 7 commits into from
Jun 5, 2017
Merged

Add juicenet platform #2670

merged 7 commits into from
Jun 5, 2017

Conversation

jesserockz
Copy link
Member

Description:
Add documentation for Juicenet platform

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

@fabaff fabaff added the new-integration This PR adds documentation for a new Home Assistant integration label May 20, 2017
Copy link
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Looks good but I would prefer changing the capitalisation of Charging station.

---
layout: page
title: "Juicenet"
description: "Instructions how to setup WiFi-equipped Juicenet Charging stations with Home Assistant."
Copy link
Contributor

Choose a reason for hiding this comment

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

Captial C in charging station shouldn't​be capitalized.

Copy link
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Looks good and can be merged after parent pr is merged.

Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

Thanks 🐦

@fabaff fabaff merged commit 4872dfc into home-assistant:next Jun 5, 2017
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