-
-
Notifications
You must be signed in to change notification settings - Fork 30k
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
Fixes for tesla. New sensors. #12225
Conversation
@@ -10,7 +10,7 @@ | |||
from homeassistant.components.sensor import ENTITY_ID_FORMAT | |||
from homeassistant.components.tesla import DOMAIN as TESLA_DOMAIN | |||
from homeassistant.components.tesla import TeslaDevice | |||
from homeassistant.const import TEMP_CELSIUS, TEMP_FAHRENHEIT | |||
from homeassistant.const import TEMP_CELSIUS, TEMP_FAHRENHEIT, LENGTH_KILOMETERS, LENGTH_MILES |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (94 > 79 characters)
Thanks! 🎉 |
So how do I get the milage&/range in kilometres instead of Miles? |
Thanks for fixing, but the fix seems to have removed the Battery level sensor. The two lines of code removed related to the Battery sensor makes the sensor update fail:
|
@JoacimRC please open an issue. |
Description: Fixes for tesla. New sensors.
Related issue (if applicable): fixes #11970
Checklist:
If the code communicates with devices, web services, or third-party tools:
tox
run successfully. Your PR cannot be merged unless tests passrequirements_all.txt
by runningscript/gen_requirements_all.py
.