Hier ein paar Mockups zu meinem Fortschritt in Bezug auf Discovery und Device-Seite. Könntet ihr mir bitte sagen, was ihr davon haltet?
Erste Entdeckung:
Wenn man zur Seite zurückkehrt (bereits erstellte Geräte werden nicht mehr angezeigt, erscheinen aber bei einer Aktualisierung auf Netatmo-Seite wieder — die Geräte sind mit Räumen verknüpft, sie müssen aktualisiert werden, wenn man z. B. ein Ventil in einen anderen Raum verschiebt — später ^^)
Ich habe allerdings noch einen ungelösten Bug. Ich habe nicht alle 16 Features im Dashboard-Editor verfügbar. Aber ich finde den Fehler nicht ^^ Zumal die Kategorien/Typen dieselben sind wie bei anderen!!
Jep ^^ aber man kann von einem Postat ausgehen => ich habe irgendwo sicher etwas übersehen ^^ Ich werde am Wochenende ein bisschen mehr recherchieren ^^
Es scheint, als ob dir 2 Features „Uhrzeit“ auf deinem Dashboard fehlen, 2 von 3… Hast du sie vielleicht einfach vergessen, zu deinem Widget hinzuzufügen?
Genau das ist das Problem ^^ Sie sind nicht auswählbar, weil sie nicht in der Auswahlliste erscheinen ^^
Der dritte Punkt ist meine Schuld ^^ Es ist der Einstellungsmodus (manuell, frostfrei,…) auswählbar (also noch nicht entwickelt).
Diese beiden Funktionen sollten normalerweise steuerbar sein. Man kann einen Start und ein Ende der manuellen Heizung definieren …
Was haltet ihr davon? Sollte ich es zunächst als schreibgeschützt einstellen und in einem späteren Pull Request den schreibgeschützten Modus mit einer Datumsbefehls-Implementierung modifizieren? Oder sollte ich es jetzt integrieren?
Ich stimme dem zu, was @pierre-gilles fordert. Wenn sie nicht schreibgeschützt sind, aber ihr Wert geändert werden kann, könnte es sein, dass das Widget „Zeitbearbeitung“ im Dashboard noch nicht entwickelt ist.
Ich empfehle, die Entwicklung des Dashboard-Widgets in einem separaten Pull Request durchzuführen, da diese Entwicklung für alle Gladys-Instanzen gilt, nicht nur für den Netatmo-Dienst.
Jetzt denke ich, dass du es so lassen kannst, da man sie nicht auf dem Dashboard sieht, weil sie nicht steuerbar sind… zumindest noch nicht… ist das so schlimm?
Kein Problem, denn man kann die Solltemperatur trotzdem steuern, es gibt einen Standardwert bei 3 Stunden.
Beim Modus ist es ähnlich: Wenn man die Solltemperatur steuert, wird der Modus auf der Backend-Seite direkt auf „manuell“ geändert (die Netatmo-API verlangt, dass man dies zwingend tut). Man kann nur nicht den Rückwechsel in den Modus „Programm“ oder „Zeitplan“ steuern.
Perfekt Besser ist es, wenn der PR so klein wie möglich ist
Wo stehst du? Lass dich nicht in den Wahnsinn der Entwicklung reinziehen, aha, sobald es für ein Gerät funktioniert, machst du den PR und wir testen und mergen das schon!
Oui oui ^^ Das Hauptproblem lag vor allem bei der OAuth2-Verbindung. Aber mit Alex’ Hilfe ist das jetzt okay ^^
Für den Rest geht es darum, die vorgeschlagenen Funktionen zu sortieren. Sagen wir mal, ich habe alles aufgelistet (weil in meinem Kopf ist alles nützlich ^^) und werde es so präsentieren. Dann reden wir darüber und entscheiden. Ich bin fast fertig.
Nur noch ein paar WebSocket-Probleme zu lösen ^^ mit potenziellen Fehlern. Ich mache viele echte Tests, aber es ist schwer, alle zu testen.
Ein Bild ist bereit und über folgenden Link verfügbar: docker pull terdious/gladys:netatmo-integration-dev
Natürlich sind alle Tests willkommen, also schlage ich vor, dass ich mein Netatmo-Konto (privat) mit denen teile, die es gerne ausprobieren möchten, auch ohne eigenes Konto.
@Tlse-vins, @PhilippeMA, @spenceur, ihr seid herzlich eingeladen, die Fehler zu testen, solange es noch welche gibt ^^ Bis bald !!
@Will_71, ich glaube, du bist der Einzige, der einen Thermostat hat, ich lasse dir die Entscheidung, was du von der Gesamtlösung hältst.
@damalgos, ebenfalls, falls du in der Nähe vorbeikommst und etwas Zeit hast ^^
Ach ja, ich habe absichtlich ein paar Dinge nicht besonders schön gemacht … um eure Meinung dazu zu hören ^^ (Spoiler: z. B. die Buttons auf der Anmeldeseite ^^)
In der Zwischenzeit arbeite ich weiter an den Tests.
Besser ist es, wenn der PR so klein wie möglich ist