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.

Sign up for GitHub

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

Jump to bottom

Implement option to reverse relay positions #62

Closed
tpanajott opened this issue Jul 20, 2023 · 3 comments
Closed

Implement option to reverse relay positions #62

tpanajott opened this issue Jul 20, 2023 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@tpanajott
Copy link
Collaborator

Implement option for the user to "switch place" of the relays. This is especially useful if the user has installed it with the wires the wrong way around.
@tpanajott tpanajott added enhancement New feature or request good first issue Good for newcomers labels Jul 20, 2023
@gvitzi
Copy link

gvitzi commented Jul 29, 2023

If you mean that button-1 will control relay-2 and vice-versa, Then I just wanted to note that this can be achieved already (at least with Home-Assistant) since the relays are reported as switches in HA, They can be added as lights in the room and then selected in the "Detached Mode" in a reverse manner.

@tpanajott
Copy link
Collaborator Author

If you mean that button-1 will control relay-2 and vice-versa, Then I just wanted to note that this can be achieved already (at least with Home-Assistant) since the relays are reported as switches in HA, They can be added as lights in the room and then selected in the "Detached Mode" in a reverse manner.

Yes, this is what I meant :) Good workaround!

tpanajott added a commit that referenced this issue Aug 5, 2023
@tpanajott
Added more button controls. Fixed issue #37 and issue #62
8fa9f52
tpanajott added a commit that referenced this issue Aug 5, 2023
@tpanajott
Merge pull request #73 from NSPManager/devel
7ffc035 
Updated main with latest devel patches. Fixed issue #62 and issue #37
@tpanajott
Copy link
Collaborator Author

Functionality added in latest merge, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tpanajott @gvitzi