Skip to content
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

Nicer output IRac::opmodeToString for FanOnly #1612

Conversation

PtilopsisLeucotis
Copy link
Collaborator

In addition to #1610

For Fan mode return "FanOnly" instead of "fan-only", it is for more nice output to string, i.e. in Tasmota log/

Before: ..."Vendor":"HAIER_AC_YRW02","Model":-1,"Mode":"fan-only","Power":"On"...
After: ..."Vendor":"HAIER_AC_YRW02","Model":-1,"Mode":"FanOnly","Power":"On"...

crankyoldgit added a commit that referenced this pull request Sep 23, 2021
Add two outputs for Fan modes.
Use a nicer & more obvious human mode by default, & a special output which is expected for Home Assistant + Google Home.

Replaces PR #1612
@crankyoldgit crankyoldgit self-assigned this Sep 23, 2021
@crankyoldgit
Copy link
Owner

crankyoldgit commented Sep 23, 2021

Marking this a duplicate of #1613 as I want this done a slightly different way.

@PtilopsisLeucotis
Copy link
Collaborator Author

Okey, good!

@PtilopsisLeucotis PtilopsisLeucotis deleted the fix_opmode_to_string branch September 23, 2021 17:04
crankyoldgit added a commit that referenced this pull request Sep 23, 2021
* Add two outputs for Fan mode.
Use a nicer & more obvious human mode by default, & a special output which is expected for Home Assistant + Google Home.
* Unit tests
* Pretty up some of the switch statements.

Replaces PR #1612
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants