Suggested changes for pimatic 0.9 readiness

[mwittig]

Hi, ich habe einige notwendige Änderungen vorgenommen um die Lauffähigkeit der Erweiterung mit Pimatic 0.9 zu gewährleisten. Alle Änderungen sind abwärtskompatibel (der Code läuft sowohl mit der aktuellen 0.8er Release als auch mit dem 0.9er branch. Einige Anmerkungen:

• Ich habe gesehen, dass es bereits eine smartfritz-promise@0.5 gibt, allerdings funktioniert mit dieser Version bei mir die Temperaturanzeige nicht.
• In der 0.9er Version können Devices zur Laufzeit gelöscht und geändert werden. Daher ist die destroy() Methode wichtig um Hintergrund Tasks (Timer, Callouts) abzubrechen.\ Ich habe hierfür nur eine rudimentäre Implementierung angelegt, die den update timer löscht**. Hier ist aber noch Ergänzungbedarf: Laufende Callouts eines Device via Smartfritz sollten auch abbgebrochen werden!
• Pimatic 0.9 führt eine striktere Prüfung des Config-Schema durch. Daher müssen alle Keys, die zur Laufzeit erzeugt werden (z.B. customConfig) im Schema vereinbart werden. Falls die Keys nur transient genutzt werden sollen, musst Du Dir eine Kopie der Config für die Verwendung im Code erstellen, z.B. via lodash.cloneDeep()
• Sweetpi wird heute noch eine ausführliche Anleitung für Plugin-Entwickler veröffentlichen. Darin ist alles noch mal ausführlich beschrieben

[andig]

Schau ich mir in Ruhe an- vielen dank! Hast du zu dem 0.5er Fehler weitere Details?

Viele Grüße, Andreas

Am 17.04.2016 um 16:29 schrieb Marcus Wittig notifications@github.com:

Hi, ich habe einige notwendige Änderungen vorgenommen um die Lauffähigkeit der Erweiterung mit Pimatic 0.9 zu gewährleisten. Alle Änderungen sind abwärtskompatibel (der Code läuft sowohl mit der aktuellen 0.8er Release als auch mit dem 0.9er branch. Einige Anmerkungen:

Ich habe gesehen, dass es bereits eine smartfritz-promise@0.5 gibt, allerdings funktioniert mit dieser Version bei mir die Temperaturanzeige nicht. In der 0.9er Version können Devices zur Laufzeit gelöscht und geändert werden. Daher ist die destroy() Methode wichtig um Hintergrund Tasks (Timer, Callouts) abzubrechen.\ Ich habe hierfür nur eine rudimentäre Implementierung angelegt, die den update timer löscht**. Hier ist aber noch Ergänzungbedarf: Laufende Callouts eines Device via Smartfritz sollten auch abbgebrochen werden! Pimatic 0.9 führt eine striktere Prüfung des Config-Schema durch. Daher müssen alle Keys, die zur Laufzeit erzeugt werden (z.B. customConfig) im Schema vereinbart werden. Falls die Keys nur transient genutzt werden sollen, musst Du Dir eine Kopie der Config für die Verwendung im Code erstellen, z.B. via lodash.cloneDeep() Sweetpi wird heute noch eine ausführliche Anleitung für Plugin-Entwickler veröffentlichen. Darin ist alles noch mal ausführlich beschrieben You can view, comment on, or merge this pull request online at:

https://github.com/andig/pimatic-fritz/pull/22

Commit Summary

Fixed compatibility issue with Coffeescript 1.9 as required for pimatic 0.9, Added destroy method to cancel an scheduled update when the device is removed or updated Updated dependencies Added missing config key created at runtime via customConfig, as pimatic 0.9 implemements stricter config schema checks File Changes

M device-config-schema.coffee (4) M fritz.coffee (52) M package.json (4) Patch Links:

https://github.com/andig/pimatic-fritz/pull/22.patch https://github.com/andig/pimatic-fritz/pull/22.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub

[mwittig]

Hast du zu dem 0.5er Fehler weitere Details?

Nein, aber ich kann es gerne hier noch mal reproduzieren. Wenn ich brauchbare Infos habe, stelle ich Dir ein Ticket ein.

[andig]

Regarding the issues I had with 0.5 I'll get back to you with more information.

I've just pushed https://github.com/andig/smartfritz/commit/53f26cb97b35c8f22afdeb45679a9ddfc8b2ab5d could you check if that fixes it? If yes I'd like to update the required version.