18 Uhr perfekt. Wir machen es so. Du kannst mich über den Weg anrufen, den du möchtest. Wir richten das Ding ein und wenn du willst, können wir es zusammen durchgehen, oder ich lasse dich machen und wir reden danach nochmal ^^
Ich schicke dir einen Google Meet-Link! Bis später 
Hallo !!
@pierre-gilles, nach unserem Anruf / Review zu Beginn der letzten Woche habe ich die meisten Änderungen / Korrekturen fast abgeschlossen:
-
Netatmo-Gesamtansicht, Rechtschreibung und Logik der Menübegriffe sowie redundante Symbole korrigiert
-
Einstellungen-Ansicht, Erstellung einer Abmeldefunktion sowie des zugehörigen Befehls im Frontend
-
Einstellungen-Ansicht, Ladebildschirm während der Verbindung
-
Einstellungen-Ansicht, Klare Anzeige, ob Gladys mit der API verbunden ist oder nicht: Dauerhafte Anzeige für beide Fälle
-
Einstellungen-Ansicht, Platzhalter „Object Object“ der zu füllenden Felder der Einstellungsansicht übernehmen
-
Entdeckungsansicht, Hinzufügen einer Nachricht, wenn nichts anzuzeigen ist, weil nicht verbunden oder keine Informationen erhalten oder nicht konfiguriert
-
Entdeckungsansicht,
-
Dashboard-Ansicht: In Arbeit, spezifische Zeilen für Netatmo von den gemeinsamen Boxen trennen (Erledigt). Rückmeldungen grauen und kennzeichnen, die später ebenfalls Befehle sein werden (Kommentare in « title » hinzugefügt)
-
Dashboard-Ansicht: Zeilen grauen und kennzeichnen, wenn Netatmo getrennt ist.
In Bild
-
Dashboard-Ansicht: Kategorie/Typ, die „undefined“ war, übernehmen und Symbole ändern, die nicht aussagekräftig genug waren.
Von unserer Seite scheint der Dienst bereit zu sein. @pierre-gilles, welche wären die nächsten Schritte, um diesen so schnell wie möglich zu veröffentlichen?
@Tlse-vins, könntest du einen neuen Test durchführen, diesmal mit dem Docker-Image? docker pull damalgos/gladys-netatmo:netatmo
Das ist ja sauber, was du da gemacht hast!
Wie jeder andere muss ich mir auch etwas Zeit nehmen, um es zu testen.
Aber ich melde mich so schnell wie möglich wieder bei dir.
Toll! Ich arbeite morgen an Gladys, ich würde einen technischen Blick auf den Code werfen, wenn du möchtest.
Danach bräuchten wir idealerweise einige Tester, um „funktionale“ Rückmeldungen zu erhalten.
Sobald die technische Überprüfung und die Tests abgeschlossen sind, können wir zusammenführen
Mit großem Vergnügen!!
Schwieriger. Wir haben 2 vollständig getestete Installationen. @Tlse-vins testet erneut, sobald er etwas Zeit hat, und @spenceur hat im Moment nicht viel Zeit. Bisher haben wir keine weiteren Personen mit Netatmo gefunden…!!
Super
Vielen Dank für deine Antworten
Ich habe am Wochenende ein bisschen zwischen meinen Arbeiten für das Kinderzimmer des zukünftigen Babys geschaut.
Ich werde versuchen, so schnell wie möglich einen Rückmeldung zu geben, falls das Ihnen weiterhelfen kann.
viel Glück 
, ist es das erste?
Ja ^^
Im Moment braucht vor allem Mama noch etwas Mut 
Aber für den Rest wird mir das CD schon helfen xD
Du würdest @VonOx unterstützen, ohne ihn zu entmutigen, oder?
Bist du dir wirklich sicher? Ich erinnere mich noch an die Stimmungsschwankungen, als wäre es gestern gewesen 
Ich höre jetzt mit dem Off-Topic auf…
Du wirst sehen, das verändert das Leben 
Da habe ich echt Glück (es sind jetzt 5 Monate und außer diesen Übelkeiten hat sie keine anderen Veränderungen [sie hatte schon immer einen dicken Kopf 
aber psst, ich habe nichts gesagt ])
:pfeifend: …
Viel Gutes für dich in Zukunft
Hallo zusammen!
Ich habe heute Morgen eine Review der PR gemacht. (Meine Review)
Funktional gesehen ist es cool, wie wir es mit @Terdious besprochen hatten, es erfüllt seinen Zweck. Ich habe keine Anmerkungen dazu gemacht.
Allerdings, was die Codequalität und die PR im Allgemeinen betrifft, denke ich, dass hier noch ein großer Reinigungsaufwand nötig ist, bevor es in die Produktion gehen kann.
Die PR Netatmo fügt 12.900 Codezeilen hinzu, das ist ein Rekord und eher beunruhigend Zum Vergleich: Die PR MQTT umfasste insgesamt 1.000 Codezeilen, und das war schon eine schöne PR.
Ich habe die PR nur diagonal überflogen, um zu sehen, was mich schockiert, und das hat mich 2 Stunden gekostet ^^ Ich habe den Code noch nicht im Detail gelesen.
Was ich mitnehme:
- Es gibt viel „toten“ Code/Konfiguration, der aus euren Experimenten vom Anfang stammt, der muss entfernt werden. Bei den NPM-Abhängigkeiten achten wir bei Gladys sehr darauf, so leicht wie möglich zu sein, und hier gibt es viele ungenutzte Abhängigkeiten.
- Die PR betrifft unzählige Dinge, das ist nicht normal. Die PR sollte den Netatmo-Ordner, das Netatmo-Frontend und ein wenig die Dashboard-Komponenten für die Anzeige betreffen, aber nicht mehr. Hier betrifft es die MQTT-Integration, einige interne Routen, das ist nicht normal.
- In einigen Dateien sind ESLint-Regeln deaktiviert. Klar, das macht die Tests grün, aber wenn es ESLint-Regeln gibt, dann gibt es dafür einen Grund. Diese eslint-disable müssen entfernt werden, und der Code muss angepasst werden.
Ich denke, mit all meinen Kommentaren habt ihr genug, um die PR zu entwirren und ihre Größe zu reduzieren.
Auf jeden Fall nochmal vielen Dank an @Terdious und @damalgos für die Arbeit. Ich gebe diese Anmerkungen, um die Qualität der PR zu verbessern und um einen leichten, leistungsfähigen und in Zukunft wartbaren Service zu haben. Nehmt diese Anmerkungen nicht persönlich, es geht darum, die Codequalität von Gladys zu sichern.
Hallo,
Ich werde mir das mal anschauen, tatsächlich werden viele Libs entfernt, das haben wir tatsächlich vergessen.
Danke für die Überprüfung, ich schaue mir das an
Hallo, ich nehme mir kurz Zeit, um zu testen:
Ich habe den folgenden Befehl auf meinem Rechner ausgeführt:
docker run -d \
--log-opt max-size=10m \
--restart=always \
--privileged \
--network=host \
--name gladys-netatmo \
-e NODE_ENV=production \
-e SERVER_PORT=1080 \
-e TZ=Europe/Paris \
-e SQLITE_FILE_PATH=/var/lib/gladysassistant/gladys-netatmo.db \
-v /var/run/docker.sock:/var/run/docker.sock \
-v /var/lib/gladysassistant:/var/lib/gladysassistant \
-v /dev:/dev \
-v /run/udev:/run/udev:ro damalgos/gladys-netatmo:latest
Ich sehe, dass der Port 1080 geöffnet ist, aber ich habe das Gefühl, dass ich nicht auf Gladys zugreife, wenn ich localhost:1080 eingebe.
Ideen?
Hallo, führe ein docker ps aus, um die Container aufzulisten
ich bestätige, dass mein Container gestartet ist. Ich sehe ihn über Kitematic als UP!
Trotzdem danke @Jean34
Gibt dir Kitematic nicht die Zugriffs-URL?