-
Notifications
You must be signed in to change notification settings - Fork 1
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
Changes to the plugin #3
Comments
I'm almost ready with the update. I'll publish a new version today. |
Sounds good. As said, I have been on the attached changes for over 2 years without any issues. |
Can you attach your file? (it was not included in the first post) |
also just emailed. something is messing up github :-( |
Probably I should have used your version as a starting point for the fixes ... I see you also made it backwards compatible with older Domoticz versions. Nice! I will add that part of the code. |
It was a bit of a hack. Not my best work haha
Kind regards,
Herman Eggink
Sent from mobile. Excuse any typos.
…________________________________
From: Lokonli ***@***.***>
Sent: Sunday, May 19, 2024 5:19:19 PM
To: lokonli/slide-domoticz-local ***@***.***>
Cc: heggink ***@***.***>; State change ***@***.***>
Subject: Re: [lokonli/slide-domoticz-local] Changes to the plugin (Issue #3)
Probably I should have used your version as a starting point for the fixes ...
I see you also made it backwards compatible with older Domoticz versions. Nice! I will add that part of the code.
—
Reply to this email directly, view it on GitHub<#3 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AA6NSF3MH45NDOZK65PS6STZDC7HPAVCNFSM6AAAAABH6HQJOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGI3TEOJZGY>.
You are receiving this because you modified the open/close state.Message ID: ***@***.***>
|
I've merged your changes (I think):
If I've missed something please let me know. |
Hey Sander,
I have geen using this for quite some time (years) but I think I made some changes. These are attached in case they are of use.
H
The text was updated successfully, but these errors were encountered: