Test Adapter Bayernluft 2.0.x
-
@mcm1957 Ich hab einen umfangreichen PR gestellt: https://github.com/iobroker-community-adapters/ioBroker.bayernluft/pull/83
Wenn du den mergen magst, kann der nochmal ausgiebig getestet werden (ich hab das schon ein paar Tage ohne Probleme im Einsatz), dann können wir auch mal das Latest angehen.Es gibt einen Breaking Change zu den bisherigen Versionen, es gibt nämlich eine komplett neue Datenpunkt-Struktur und es wird auch eine neue export-Datei auf den Geräten selbst benötigt. Für die Zukunft wird es dann aber wie schon beschrieben einfacher für neue Nutzer.
closed Redesign of the adapter (and fixed linting) #83
-
@boriswerner said in Test Adapter Bayernluft 2.0.x:
Hab dir schon im PR geantwortet. Hoff ich komm die Woche dazu. Da ja jedenfalls auch der Review fürs Repo offen ist kann man das auf einmal machen. Ev. sind ja diesbezüglich noch Änderungen notwendig.
Bitte häng mal im PR einen Objectdump (siehe https://github.com/ioBroker/ioBroker.repochecker/blob/master/OBJECTDUMP.md) an.
-
@boriswerner sagte in Test Adapter Bayernluft 2.0.x:
kann der nochmal ausgiebig getestet werden
ist dies in der 2.0.0-alpha.0 schon aufgenommen, oder ist noch was neues implementiert?gefunden
Breaking Change: All states from the 2.alpha versions or version 1 of the adapter were removed -
@negalein Es gibt eine komplett neue Datenpunktstruktur. An neuen Features ist "nur" die Korrektur der Dezimalstellen (siehe Kommentar von Wildbill) und das individuelle Setzen der einzelnen Lüfter bei ausgeschaltetem Gerät (ebenfalls von Wildbill eingebracht). Die Datenpunkte sind jetzt bzgl. der Rollen und Datentypen optimiert und insgesamt auch das Verhalten im Hintergrund wurde optimiert (check, ob ein Gerät überhaupt online ist, Fehlerbehandlung, etc.).
Also viel im Hintergrund aber auch ein bisschen neues -
@negalein
Nein@boriswerner hat einen größeren Umbau vorgenommen. Ich werde mir diesen im Laufe der Woche ansehen - wenn ich dazu komme. Derzeit ist diese Änderung noch nirgends in einem offiziellen Repo / im community Github enthalten. Da er die Struktur der States geändert hat wird was wohl eine 3.x.x werden müssen. Aber bevorich da eine Release einzeihe muss ich schaun was da wie geänder wird. Insbesondere warte ich auf den Dump der Objekte.
@Negalein
Da du wahrscheinlich noch die alte Version installiert hats, könntest du mir bitte auch einen Dump der jetzigen Struktur zum Vergleich zukommen lassen? Einfach hier oder an den PR anhängen. Anleitung siehe https://github.com/ioBroker/ioBroker.repochecker/blob/master/OBJECTDUMP.md.P.S:
Ich gehe davon aus dass die Lüfter LOKAL sind und daher in den Objekten keinerlei sicherheitsrelevante Daten drinnen sind. Aberbitte dies checken und diese ggf ausnullen/maskieren. -
@mcm1957 mein Dump ist hochgeladen. Die Geräte sind lokal, es sind MAC und die lokale IP enthalten, halte ich aber nicht für sicherheitsrelevant (oder?)
-
@boriswerner
Die IP sicher lokal (oder besitzt du ein öffentliche IPv4 Subnetz
). Die MAC würde ich auch nicht als kritisch sehen.
DANKE -
@mcm1957 sagte in Test Adapter Bayernluft 2.0.x:
bitte auch einen Dump der jetzigen Struktur zum Vergleich zukommen lassen?
-
Sorry fürchte wird die Woche nichts mehr mit Review. Hab mir Darminfekt eingefangen u lieg zZ flach
-
@mcm1957 Kurier Dich aus. Gute Besserung.
Gruss, Jürgen
-
Zuerst mal an dieser Stelle ein herzliches DANKE an @Boriswerner für die Anpassungen und Erweiterungen des Adapters.
Ich habe jetzt mal:
- die Änderungen von boriswerenr gemerged
- ein standardmäßiges Review für LATEST / BETA Repository durchgeführt (sie(he PR)
- diverse Anpassungen auf Grund des Reviews implementiert (siehe PR)
Das Resultat ist mal eine erste Testreleae 3.0.0-alpha.0.
Für den Test / Feedback habe ich mal einen 3.0.x Thread eröffnet:
https://forum.iobroker.net/topic/80006/test-adapter-bayernluft-3-0-x-alpha-testBitte dort weitermachen.
P.S: Die restlichen offenen Punkte werde ich hoffentlich zeitnahe erledigen.
