ioBroker / ioBroker.maxcul

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

Pairmode #33

Closed StenmannsAr closed 5 years ago

StenmannsAr commented 5 years ago

like in #33 suggested. I created a user enabled pairing mode. When the user set the value to true. The cul is for 30 seconds in paring mode.

StenmannsAr commented 5 years ago

Bitte noch nicht mergen.

guergen1 commented 5 years ago

Was möchtest Du mir sagen? Mergen?

guergen1 commented 5 years ago

Eingefügt, keine Änderung...

guergen1 commented 5 years ago

Jetzt startet der Maxcul adapter nicht durch....

StenmannsAr commented 5 years ago

there are still some errors

guergen1 commented 5 years ago

Nach einem kompletten Reboot des Nuc´s, läuft der Adapter wieder...

guergen1 commented 5 years ago

Ob der "Schalter" true oder false ist, ist dem Maxcul egal, er nimmt den Wallthermostat auf!

StenmannsAr commented 5 years ago

Ob der "Schalter" true oder false ist, ist dem Maxcul egal, er nimmt den Wallthermostat auf!

Das ist sehr merkwürdig. Hast du das Repository geklonet oder die Änderungen per hand durchgeführt?

guergen1 commented 5 years ago

Per hand durchgeführt

guergen1 commented 5 years ago

Habe grade noch Fehler bei mir gefunden, warum ich das überlesen haben sollte ist mir unklar, habe aus dem "Files changed" bereich grade noch drei Wörter ändern müssen...

guergen1 commented 5 years ago

Ändern heisst bei mir: ab in die Zeile, copy & paste.... nicht komplett von Hand! Ich möchte ja Schreibfehler verhindern!

guergen1 commented 5 years ago

Habe jetzt noch einmal alles kontrolliert, jetzt sind älle Änderungen in meinen Files drin!

StenmannsAr commented 5 years ago

Es wäre am besten wenn du das Repository per git clone oder als Zip-Datei herutnerlädst und damit den iobroker.maxcul adapter ersetzt. Du müsstest dann darauf achten das Dann sind wir auch ganz sicher, dass du alle Änderungen hast.

https://github.com/StenmannsAr/iobroker.maxcul/archive/pairmode.zip

guergen1 commented 5 years ago

Das zip gibt´s nicht: 404

StenmannsAr commented 5 years ago

Das zip gibt´s nicht: 404

probier den Link bitte jetzt noch einmal

guergen1 commented 5 years ago

wo kann ich denn das rep per git ziehen? grafik

guergen1 commented 5 years ago

https://github.com/ioBroker/ioBroker.maxcul/pull/url

das ist die url die hinter dem Link liegt

StenmannsAr commented 5 years ago

dann copy & paste bitte den Namen des Links

guergen1 commented 5 years ago

Ich habe es... war etwas blind...

guergen1 commented 5 years ago

Habe es jetzt von hier: https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode

guergen1 commented 5 years ago

Beim Install: $ ./iobroker url "https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode" --debug install https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode/tarball/master npm install https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode/tarball/master --production --save --prefix "/opt/iobroker" (System call) npm ERR! code E404 npm ERR! 404 Not Found - GET https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode/tarball/masternpm ERR! 404 npm ERR! 404 'https://github.com/StenmannsAr/iobroker.maxcul/tree/pairmode/tarball/master' is not in the npm registry.npm ERR! 404 Your package name is not valid, because npm ERR! 404 1. name can only contain URL-friendly charactersnpm ERR! 404 2. name can no longer contain capital letters npm ERR! 404 npm ERR! 404 Note that you can also install from a

npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! A complete log of this run can be found in:npm ERR! /home/iobroker/.npm/_logs/2019-08-30T13_07_34_206Z-debug.log

got /opt/iobroker/node_modules/iobroker.maxcul/admin upload [3] maxcul.admin /opt/iobroker/node_modules/iobroker.maxcul/admin/words.js words.js application/javascript upload [2] maxcul.admin /opt/iobroker/node_modules/iobroker.maxcul/admin/maxcul.png maxcul.png image/png upload [1] maxcul.admin /opt/iobroker/node_modules/iobroker.maxcul/admin/index_m.html index_m.html text/html upload [0] maxcul.admin /opt/iobroker/node_modules/iobroker.maxcul/admin/index.html index.html text/html process exited with code 0

guergen1 commented 5 years ago

$ ./iobroker url "https://github.com/ioBroker/ioBroker.maxcul/pull/url" --debug install https://github.com/ioBroker/ioBroker.maxcul/pull/url/tarball/master npm install https://github.com/ioBroker/ioBroker.maxcul/pull/url/tarball/master --production --save --prefix "/opt/iobroker" (System call) npm ERR! code E404 npm ERR! 404 Not Found - GET https://github.com/ioBroker/ioBroker.maxcul/pull/url/tarball/masternpm ERR! 404 npm ERR! 404 'https://github.com/ioBroker/ioBroker.maxcul/pull/url/tarball/master' is not in the npm registry. npm ERR! 404 Your package name is not valid, because npm ERR! 404 1. name can only contain URL-friendly characters npm ERR! 404 2. name can no longer contain capital letters npm ERR! 404 npm ERR! 404 Note that you can also install from anpm ERR! 404 tarball, folder, http url, or git url.

npm ERR! A complete log of this run can be found in:npm ERR! /home/iobroker/.npm/_logs/2019-08-30T13_10_42_585Z-debug.log

process exited with code 0

StenmannsAr commented 5 years ago

1) Module Instanz von iobroker.maxcul deaktivieren 2) den Ordner iobroker.maxcul aus dem Ordner /opt/iobroker/node_modules löschen 3) Das ZIP-File entpacken 4) Den entpackten Ordner iobroker.maxcul nach /opt/iobroker/node_modules kopieren 5) Instanz wieder starten

guergen1 commented 5 years ago

OK, hatte nix gelöscht.... moment

guergen1 commented 5 years ago

Files sind da, ich hoffe dass alles passt wegen der Berechtigungen unter Linux

StenmannsAr commented 5 years ago

ich sehe grade der entpackte Ordner heißt iobroker.maxcul-pairmode bitte in iobroker.maxcul umbenennen. Es tut mir wirklich leid, aber es müssen wohl auch die Berechtigungen angepasst werden.

in einem temrinal chmod 777 /opt/iobroker/node_modules/iobroker.max/* Der einfachheit halber hat damit jeder auf deinem nuc system Zugriff auf den Ordner iobroker.maxcul Bitte nach dem Test den Ordne 4) Den entpackten Ordner iobroker.maxcul nach /opt/iobroker/node_modules kopierenr löschen.

guergen1 commented 5 years ago

Aha... jetzt Paimode true und er liesst es ein, nach gefühlt 30 sek liesst er es nicht ein. Das geht schon mal. aber true bleibt unter info stehen, geht nicht sofort weg

guergen1 commented 5 years ago

Ich habe den originalen ordner kopiert und den inhalt deines Ordners kopiert, 777 haben jetzt alle files!

guergen1 commented 5 years ago

Ich kann nur nicht den Beitzer des Ordners anpassen, aber das muss mit 777 ja auch nicht mehr gemacht werden!

guergen1 commented 5 years ago

grafik grafik

guergen1 commented 5 years ago

Linux ist nicht ganz so meins, auf dem Raspi war das etwas einfachen, habe hier jetzt unbuntu...

guergen1 commented 5 years ago

Pairmode steht unter info mmer noch mit true, iobroker interessiert es aber nciht... :-)

guergen1 commented 5 years ago

grafik

guergen1 commented 5 years ago

grafik

StenmannsAr commented 5 years ago

Erst einmal danke, dass du dir die ganze Mühe machst.

Bitte Zeile 83 und 88 in der main.js mit der Zeile ersetzten:

adapter.setState('info.enablePairingMode',false,true);

guergen1 commented 5 years ago

Ist ja zu meinem Vorteil :-) Zeile ist angepasst!

guergen1 commented 5 years ago

Jetzt steht es auf false und lässt aber das pairing zu!

guergen1 commented 5 years ago

OK, einmal von hand auf true und wieder auf false, läuft

StenmannsAr commented 5 years ago

super dann einfach ein wenig damit rumspielen und wenn alles wie erwartet läuft dein OK geben.

Bitte lösch den Ordner nach deinen Tests wieder (oder schau das nur der iobroker rechte auf den Ordner hat) und warte auf die Aktualisierung des Adapters.

Vielen Dank

guergen1 commented 5 years ago

OK, bin schopn dabei; auch die Rechte wieder anzupassen!

guergen1 commented 5 years ago

Alles scheint wie gewollt zu funktionieren: stellt man es auf true, liesst er die Daten ein, stellt man es auf false macht er es nicht. nach einer Zeit, ich denke 30 sek. springt das "enablePairingMode" zurück auf false! Perfekt!

StenmannsAr commented 5 years ago

Sehr schön.

@Apollon77 @GermanBluefox Dann könnte jetzt der Pull-Request gemerged werden.

Apollon77 commented 5 years ago

Es fehlen noch die News in io-package ... (Reminder for myself or someone who is faster - https://translator.iobroker.in/ can be used to get translations) ;-)

StenmannsAr commented 5 years ago

done :-)

Apollon77 commented 5 years ago

thank you!

Apollon77 commented 5 years ago

published to npm