Skip to content

add HTU21D sensor platform doc#2820

Merged
fabaff merged 3 commits into
home-assistant:nextfrom
azogue:sensor-htu21d
Jun 22, 2017
Merged

add HTU21D sensor platform doc#2820
fabaff merged 3 commits into
home-assistant:nextfrom
azogue:sensor-htu21d

Conversation

@azogue
Copy link
Copy Markdown
Member

@azogue azogue commented Jun 15, 2017

Description:

New HTU21D temperature and humidity sensor platform documentation

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

@Landrash Landrash added the new-integration This PR adds documentation for a new Home Assistant integration label Jun 16, 2017
Copy link
Copy Markdown
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.

Minor changes are needed but looks good in general.


It will output a table like this:

```bash
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This isn't bash. Suggest you use text for highlighting.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

# Example configuration.yaml entry
sensor:
- platform: htu21d
name: Ambient
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Name can be removed from the example since it's an optional variable.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've changed the customize section too to use the default root name

@fabaff fabaff dismissed Landrash’s stale review June 22, 2017 07:03

Comments addressed.

Copy link
Copy Markdown
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 3881bff into home-assistant:next Jun 22, 2017
@azogue azogue deleted the sensor-htu21d branch July 8, 2017 13:03
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