ioBroker / ioBroker.maxcul

Control max! devices over CUL stick
Other
4 stars 8 forks source link

Ermöglichen der Abschaltung des Pairing modus? #32

Closed StenmannsAr closed 5 years ago

StenmannsAr commented 5 years ago

Hallo Zusammen,

aus gegeben Anlass #30 würde ich gerne darüber diskutieren, wie sinnvoll es wäre den pairing modus nicht auf dauerhaft aktiv zu schalten, sondern dem Benutzer die Wahl zu lassen wann er aktiviert sein soll?

Was haltet Ihr davon?

Apollon77 commented 5 years ago

Other adapters also do that. You could have a state to start pairing mode for e.g. 30s and then it stops again.

guergen1 commented 5 years ago

Die Idee finde ich sehr gut!

guergen1 commented 5 years ago

Können wir uns mal austauschen zwecks Funktion mit dem Wall-Thermostat?

Ralf

Von: StenmannsAr notifications@github.com Gesendet: Dienstag, 27. August 2019 09:02 An: ioBroker/ioBroker.maxcul ioBroker.maxcul@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Betreff: [ioBroker/ioBroker.maxcul] Ermöglichen der Abschaltung des Pairing modus? (#32)

Hallo Zusammen,

aus gegeben Anlass #30 https://github.com/ioBroker/ioBroker.maxcul/issues/30 würde ich gerne darüber diskutieren, wie sinnvoll es wäre den pairing modus nicht auf dauerhaft aktiv zu schalten, sondern dem Benutzer die Wahl zu lassen wann er aktiviert sein soll?

Was haltet Ihr davon?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.maxcul/issues/32?email_source=notifications&email_token=AIGNI3OMPMAHJ5HC7WDB7CDQGTGPJA5CNFSM4IP64LZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HHSBBHQ , or mute the thread https://github.com/notifications/unsubscribe-auth/AIGNI3NM62PUVMIHTLAUEJDQGTGPJANCNFSM4IP64LZA .


Diese E-Mail wurde von Avast Antivirus-Software auf Viren geprüft. https://www.avast.com/antivirus

StenmannsAr commented 5 years ago

Guten Abend,

ich habe das einmal implemeniert. Mir fehlt leider im Moment die Zeit das Feature ausgiebig zu testen. Ist im pull Request #33

@guergen1 Wenn du das genauer testen könntest, wäre ich dir sehr dankbar.

guergen1 commented 5 years ago

Werde ich machen!

StenmannsAr commented 5 years ago

super, Danke.

Bei Problemen oder Fragen stehe ich gerne zu Verfügung.

Am besten nimmst du mein Repo mit dem Pairing Branch.

https://github.com/StenmannsAr/ioBroker.maxcul/tree/pairmode

guergen1 commented 5 years ago

Habe die entsprechenden Zeilen geändert. Ein "enable Pair-Mode" state wurde generiert. und steht auf "false" habe ihn auf true und nach einer Zeit wieder auf false gesetzt. Leider wird nach 3 Sekunden der Thermostat automatisch eingefügt wenn ich auf dem Thermostat den Pairing-Mode aktiviere!

grafik grafik

StenmannsAr commented 5 years ago

Das liegt daran, dass das Abschalten durch manuelles setzten von false nicht implementiert war. Der Fix steht in meinem Repo bereit.

https://github.com/ioBroker/ioBroker.maxcul/compare/master...StenmannsAr:pairmode