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

New switchmodes TOGGLEMULTI, FOLLOWMULTI and FOLLOWMULTI_INV #7522

Merged
merged 2 commits into from
Jan 16, 2020
Merged

New switchmodes TOGGLEMULTI, FOLLOWMULTI and FOLLOWMULTI_INV #7522

merged 2 commits into from
Jan 16, 2020

Conversation

to-scho
Copy link

@to-scho to-scho commented Jan 15, 2020

Description:

This may solve #7519 as enhancement

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest dev branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR.
  • The code change is tested and works on core 2.6.1
  • The code change pass travis tests. Your PR cannot be merged unless tests pass
  • I accept the CLA.

@to-scho to-scho requested a review from arendst January 15, 2020 12:31
typos
@arendst arendst merged commit 251ea08 into arendst:development Jan 16, 2020
@to-scho to-scho deleted the feature/new_multi_switchmodes branch January 16, 2020 09:27
arendst added a commit that referenced this pull request Jan 16, 2020
Add ``SwitchMode 8`` ToggleMulti, ``SwitchMode 9`` FollowMulti and ``SwitchMode 10`` FollowMultiInverted (#7522)
@jensuffhaus
Copy link

@to-scho @arendst

I have just upgraded my one Sonoff Mini to the latest development build and my testing so far shows the multi toggle feature is working, at least a double toggle. A triple toggle results in a double toggle mqtt message and then a local relay toggle.

A future enhancement may be to allow infinite toggles all with the 500ms gap and have the toggle count as the mqtt payload but for now the double toggle will suffice.

This is a great feature for light switch automation and will allow so much existing old hardware to be used as smart input.

Thanks @to-scho for the quick response.

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.

3 participants