Hannes

> > I will add a Configurable Time to the Worker aswell as "Enable and Disable" Option :) > > One value is sufficient. Negative or zero means "disable". Done...

> @Sn0w3y Thank you so much! We’ve started testing it internally! Does it work? Could you please tell us if the PR is successful ?

> thx @michaelgrill for the review > > > * Add a different warning channel if authentication fails, documentation is good but not everyone may read it. > > I...

Theoretically this would be possible. But since Version 2 of Shelly Devices they switched to [HTTP Digest Authentication](https://datatracker.ietf.org/doc/html/rfc7616) so it would be a lets call it "Nightmare" to add this...

> Yes, everything in that direction is welcome! Ideally we would unify these State-Channels in a common Nature for all Shelly implementations. > > What about [#3203](https://github.com/OpenEMS/openems/pull/3203). Should I merge...

> I can simply close #3203 if you want. As I suggested i guess you do not need to close it, just add another "check" for "isPasswordSet" or something like...

@sfeilmeier i created a new PR addressing my suggetions and your approach unifying these State-Channels in a common Nature for all Shelly implementations.

> @Sn0w3y looking at your new PR #3248 I should close my PR (or revert the commit at least) now? I will/would need some support in adding the check for...

> @Femumme Of course we are interested in contributions, but reviewing is also a lot of voluntary spare-time work... > > @Sn0w3y Could you do a first review maybe? @sfeilmeier...

> > That's good to know! I understand that you guys got a lot on your plate. What can I do to make the review easier for you? > >...