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

[Shutter] Fix PowerON #22548

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

[Shutter] Fix PowerON #22548

wants to merge 1 commit into from

Conversation

SteWers
Copy link
Contributor

@SteWers SteWers commented Nov 25, 2024

Description:

I have on some devices, not on all, the problem, that both shutter relais are switches on after power on. This doesn't happen after a software restart. This PR fixes the problem.
@stefanbode: Please have a look at it, if I did it the right way.

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works with Tasmota core ESP8266 V.2.7.8
  • The code change is tested and works with Tasmota core ESP32 V.3.1.0.241117
  • I accept the CLA.

@stefanbode
Copy link
Contributor

Let’s keep this on hold. Here you disable the power switch on restart. May be a good idea if shutters defined and a shutter relay but should not kick in in any other situation. I do not understand why this happens NOW. It was not an issue the last 5 years.

@SteWers
Copy link
Contributor Author

SteWers commented Nov 26, 2024

I think, that this happens since a few month. After a power outage, I recognised it, but I thought it was caused by my hardware module. While testing the UI fix I saw this on the test device (D1 mini with only a shutter configured). On other devices it does not happen. Very crazy. Will do some further tests.

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