-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fix: "Use metric units" toggle has no effect and simplify sensor creation #219
Conversation
Hi @DurgNomis-drol, it would be great if we could merge this before adding new sensors (e.g. #222) as it includes some significant improvements to the sensor creation. Especially when dealing with metric vs non-metric units. 😉 |
Hi, @DurgNomis-drol, can you please take a look and approve if the changes are ok for you. 😊 |
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.
Awesome thanks! Sorry for the long wait
Fix: #217 and add improements to sensor creation. Especially when dealy with metric vs non metric units.