Skip to content
This repository was archived by the owner on Jun 23, 2022. It is now read-only.

HA HB synchronization display mode change#232

Merged
robbiet480 merged 4 commits intohome-assistant:masterfrom
roiff:master
Oct 26, 2017
Merged

HA HB synchronization display mode change#232
robbiet480 merged 4 commits intohome-assistant:masterfrom
roiff:master

Conversation

@roiff
Copy link
Contributor

@roiff roiff commented Oct 4, 2017

HA HB synchronization display mode change

roiff added 2 commits October 4, 2017 13:26
 Simultaneous display for change mode on HA
@homeassistant
Copy link

Hi @roiff,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

}
HomeAssistantClimate.prototype = {
onEvent: function (oldState, newState) {
const list = {'idle':0, 'heat':1, 'cool':2, 'auto':3, 'off':0}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A space is required after '{' object-curly-spacing
Unnecessarily quoted property 'idle' found quote-props
Missing space before value for key 'idle' key-spacing
Unnecessarily quoted property 'heat' found quote-props
Missing space before value for key 'heat' key-spacing
Unnecessarily quoted property 'cool' found quote-props
Missing space before value for key 'cool' key-spacing
Unnecessarily quoted property 'auto' found quote-props
Missing space before value for key 'auto' key-spacing
Unnecessarily quoted property 'off' found quote-props
Missing space before value for key 'off' key-spacing
A space is required before '}' object-curly-spacing
Missing semicolon semi

@robbiet480 robbiet480 merged commit fe6c4cf into home-assistant:master Oct 26, 2017
@robbiet480
Copy link
Contributor

Thanks! 🐬 🍪 💯

@roiff
Copy link
Contributor Author

roiff commented Nov 10, 2017

I am Chinese, I saw your article in China today, about the translation app, what can help you?

@robbiet480
Copy link
Contributor

Hi @roiff that was for the iOS app, it's been fully translated now, thank you!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants