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

Remove units for humidity in Wundeground sensor #3018

Merged
merged 2 commits into from
Aug 29, 2016
Merged

Remove units for humidity in Wundeground sensor #3018

merged 2 commits into from
Aug 29, 2016

Conversation

arsaboo
Copy link
Contributor

@arsaboo arsaboo commented Aug 28, 2016

Description: Wunderground returns the humidity with % so need to remove the extra % Otherwise, it appears as
image

Here's the current entity information...note that the state already has %.
image

Wunderground returns the information with the units.
@mKeRix
Copy link
Contributor

mKeRix commented Aug 28, 2016

Shouldn't we rather strip the % from the number instead of removing the unit in HASS?

@robbiet480
Copy link
Member

Let's get this into the upcoming 0.27.1. I agree with @mKeRix that we should remove the % from the number instead of removing the unit.

@arsaboo
Copy link
Contributor Author

arsaboo commented Aug 28, 2016

Done...removed the % from the return value and retained the units.

@Teagan42
Copy link
Contributor

🐬

@Teagan42 Teagan42 merged commit 39402af into home-assistant:dev Aug 29, 2016
@Teagan42 Teagan42 deleted the patch-2 branch August 29, 2016 03:05
@robbiet480 robbiet480 mentioned this pull request Aug 30, 2016
@home-assistant home-assistant locked and limited conversation to collaborators Mar 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants