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 'nomapnt', 'outcurnt', 'loadapnt' fields #16176

Merged
merged 2 commits into from
Sep 2, 2018

Conversation

MarcSN311
Copy link
Contributor

Added 'nomapnt', 'outcurnt', 'loadapnt' fields and the correspoding Units Ampere and Volt-Ampere to INFERRED_UNITS.

@homeassistant
Copy link
Contributor

Hi @MarcSN311,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@homeassistant homeassistant added merging-to-master This PR is merging into the RC branch and should probably change the branch to `dev`. platform: sensor.apcupsd small-pr PRs with less than 30 lines. cla-needed labels Aug 24, 2018
@ghost ghost added the in progress label Aug 24, 2018
@@ -49,6 +49,7 @@
'linefreq': ['Line Frequency', 'Hz', 'mdi:information-outline'],
'linev': ['Input Voltage', 'V', 'mdi:flash'],
'loadpct': ['Load', '%', 'mdi:gauge'],
'loadapnt' : ['Load Apparent Power', '%', 'mdi:gauge'],

Choose a reason for hiding this comment

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

whitespace before ':'

@MartinHjelmare
Copy link
Member

Please rebase and change target branch to dev.

@fabaff fabaff changed the base branch from master to dev August 24, 2018 21:22
@fabaff fabaff added docs-missing and removed merging-to-master This PR is merging into the RC branch and should probably change the branch to `dev`. labels Aug 24, 2018
Also added Ampere and Volt-Ampere to INFERRED_UNITS
@MarcSN311
Copy link
Contributor Author

I rebased. What does the "docs-missing" label mean?

@MartinHjelmare
Copy link
Member

There were instructions about that in the PR template.

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.

🐦

@fabaff fabaff merged commit 357e5ea into home-assistant:dev Sep 2, 2018
@ghost ghost removed the in progress label Sep 2, 2018
girlpunk pushed a commit to girlpunk/home-assistant that referenced this pull request Sep 4, 2018
* Added 'nomapnt', 'outcurnt', 'loadapnt' fields

Also added Ampere and Volt-Ampere to INFERRED_UNITS

* Fix lint issue
@balloob balloob mentioned this pull request Sep 17, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants