Skip to content

Allow choosing Celsius or Farenheit for temp#4

Open
akileos wants to merge 1 commit intoHomeAutomationForGeeks:masterfrom
akileos:master
Open

Allow choosing Celsius or Farenheit for temp#4
akileos wants to merge 1 commit intoHomeAutomationForGeeks:masterfrom
akileos:master

Conversation

@akileos
Copy link
Contributor

@akileos akileos commented Aug 21, 2015

No description provided.

@HomeAutomationForGeeks
Copy link
Owner

It's a good change, but it takes away the option to display both (which is how I prefer it, to become more familiar with both standards). Maybe add a third option - like "b" for "both", or "cf" for "celsius/fahrenheit" - which keeps things as they were before?

"c" can stay the default though as that's what the vast majority of countries use.

@akileos
Copy link
Contributor Author

akileos commented Aug 22, 2015

Will check when I can do that, no problem, just an elseif block :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants