Skip to content

Updated documentation with new sensor #4094

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 12 commits into from
Dec 24, 2017
Merged

Conversation

jbarrancos
Copy link
Contributor

@jbarrancos jbarrancos commented Nov 29, 2017

Description:

Added rainsensor

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

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.

@jbarrancos jbarrancos mentioned this pull request Nov 29, 2017
8 tasks
@jbarrancos jbarrancos changed the title Updated docuementation with new sensor Updated documentation with new sensor Nov 29, 2017
@frenck frenck added the new-integration This PR adds documentation for a new Home Assistant integration label Dec 1, 2017
footer: true
logo: rainbird.png
ha_category: Hub
ha_release: 0.57
Copy link
Member

Choose a reason for hiding this comment

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

Needs to updated after the merging in the main repo.

```yaml
rainbird:
host: '1.1.1.1'
password: 'XXXXXXX'
Copy link
Member

Choose a reason for hiding this comment

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

Please use two spaces for indent.

sensor:
- platform: rainbird
monitored_conditions:
- rainsensor
Copy link
Member

Choose a reason for hiding this comment

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

Wrong indent.

Copy link
Contributor

Choose a reason for hiding this comment

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

The line with - rainsensor needs two more spaces in front of it.

# Example configuration.yaml entry
sensor:
- platform: rainbird
monitored_conditions:
Copy link
Contributor

Choose a reason for hiding this comment

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

Indentation is off here, but lets drop this anyway and show only the minimal config.

footer: true
logo: rainbird.png
ha_category: Hub
ha_release: 0.57
Copy link
Contributor

Choose a reason for hiding this comment

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

0.61

footer: true
logo: rainbird.png
ha_category: Hub
ha_release: 0.59.3
Copy link
Contributor

Choose a reason for hiding this comment

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

0.61


Configuration variables:

- **stickip** (*Required*): The IP address of your LNK WiFi Module.
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be **host**. Looks like an mistake from the old docs.

@fabaff fabaff merged commit fd2dd4d into home-assistant:next Dec 24, 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.

4 participants