search

geraldoramos / nikola

⚡Unofficial cross-platform desktop app to monitor and control Tesla vehicles, powered by Electron & React
https://nikola.site
MIT License
149 stars 43 forks source link

Temperature is showing wrong unit #4

Closed chriscardinal closed 5 years ago

chriscardinal commented 5 years ago

First, this app looks great, well done! A welcome successor to the no-longer-maintained VisibleTesla.

The temperature is showing Celsius, but with an F:

image

Doesn't appear to be a way to change the units, either.

geraldoramos commented 5 years ago

Good call, forgot to handle that. I will fix it in the next release. Auto-update should be working fine so you should get an alert once a new version is available, it should be soon.

geraldoramos commented 5 years ago

So, looks like the API always returns celsius and it doesn't tell which scale it uses, so I assume that it's always Celsius. In the future we can maybe do a converter in the UI and make a toggle, but not sure how important it is. I've changed from F to C in the UI and this should be in the next release. Thanks for catching this @chriscardinal.

jbenamy commented 5 years ago

I unfortunately would not use the app if the units were stuck in metric. Please have option to convert to Fahrenheit.

geraldoramos commented 5 years ago

@jbenamy this has been fixed in the last release. Please download 0.2.6.