Skip to content

Added support for RSL devices sold by Conrad as Switch Type E#39

Open
glaukos wants to merge 8 commits intosui77:masterfrom
glaukos:master
Open

Added support for RSL devices sold by Conrad as Switch Type E#39
glaukos wants to merge 8 commits intosui77:masterfrom
glaukos:master

Conversation

@glaukos
Copy link
Copy Markdown

@glaukos glaukos commented Mar 15, 2016

Changes according to http://www.mikrocontroller.net/topic/252895 and https://forum.pilight.org/Thread-Fully-Supported-Conrad-RSL-Switch

I could verify to some degree with my personal blind shuttter switch.

Comment thread RCSwitch.cpp
* @param sGroup Number of the switch group (1..4)
* @param nDevice Number of the switch itself (1..4)
*/
void RCSwitch::switchOffRSL(int sGroup, int nDevice) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could you please use two spaces for indention, instead of tabs? To match the surrounding code... :-)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sure, will replace tabs. Although the surrounding code is already a mixture of two and four spaces intendations.

Comment thread RCSwitch.cpp Outdated
}

/**
<<<<<<< HEAD
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks like you had a merge or rebase conflict and did not resolve it completely?

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