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.

Sign up for GitHub

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

Jump to bottom

Wrong case on sensor units #59

Closed
arnoo opened this issue May 13, 2016 · 4 comments
Closed

Wrong case on sensor units #59

arnoo opened this issue May 13, 2016 · 4 comments

Comments

@arnoo
Copy link

arnoo commented May 13, 2016

The first letter of measurement units for sensors seems to get transformed to uppercase at some point, I'm not sure where. The unit of measurement should be exactly as set in the component code.

For example, the code of the speedtest sensor correctly specifies the unit of measurement for ping in "ms" (miliseconds). The interface displays "Ms" (megaseconds).
@fabaff
Copy link
Member

fabaff commented May 13, 2016

There is already an open story for this.

@arnoo
Copy link
Author

arnoo commented May 13, 2016

Cool, thanks :)

@fabaff
Copy link
Member

fabaff commented May 13, 2016

This doesn't mean that if anybody is working on it.

PR would be welcome.

@fabaff
Copy link
Member

fabaff commented May 13, 2016

In the Home Assistant issue tracker: home-assistant/core#938

@arnoo arnoo mentioned this issue May 30, 2016
Merged
@balloob balloob closed this as completed Jun 2, 2016
tkdrob pushed a commit to tkdrob/frontend that referenced this issue Apr 20, 2021
@robbiet480
Location Update (home-assistant#59)
f5c13ed 
Location Update
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@fabaff @arnoo @balloob