Airtable ‹ en › und ‹ fr › aktualisiert (Airtable - Netatmo NATherm1).
Allerdings sehe ich nicht, wie ich es in die Dokumentation integrieren soll, da die Dokumentations-ID ‹ netatmo › nicht existiert und ich nicht weiß, wie ich sie hinzufügen soll:
Zur Info, ich habe den Amazon-Links &tag=gladproj-21 hinzugefügt. Die Weiterleitung funktioniert, aber ich bin mir nicht sicher, ob die Affiliate-Zuordnung berücksichtigt wird (‹ 21 › = Jahr 2021? Muss man das jedes Jahr ändern?).
Danke für die Dokumentation! Sie ist sehr vollständig und sauber.
Kannst du für den Titel einen längeren Titel verwenden, der klar erklärt, was dieses Tutorial ermöglicht?
Beispiel: Netatmo-Thermostate mit der Smart-Home-Installation verbinden
Das hilft, sich bei Google mit solchen Keywords zu platzieren, mit einem ansprechenden Titel, der nicht mit Gladys verbunden ist (Gladys nicht im Titel erwähnen)
Für den Abschnitt „Roadmap“ würde ich alle Daten entfernen, denn bei Gladys gibt es keine festen Verpflichtungen, sonst entsteht nur Enttäuschung Die Prioritäten ändern sich, und es ist unmöglich, sich zu verpflichten… Einfach eine Liste der Dinge, die du machen möchtest!
Das ist nett, aber es ist nicht nötig, Affiliate-Links hinzuzufügen, sie werden automatisch eingefügt! Und nein, 21 ist nur eine Zahl, die vor langer Zeit von Amazon generiert wurde
Wie weit bist du mit der Integration und wer hat sie schließlich getestet?
Hallo @pierre-gilles!!
Und willkommen zurück, beste Wünsche für 2024.
Okay, ich mache die Änderungen für die Dokumentation ^^
Für den Fall habe ich es wie bei Sonos am Ende so übernommen: « Netatmo-Thermostate mit dem Smart Home verbinden »
Entschuldige, ich weiß, dass die Datei sehr lang zum Überarbeiten ist ^^
Also wie oben erwähnt, der PR für die Integration ist fertig,
und ich habe dir eine Review-Anfrage geschickt.
Getestet von @Will_71 (Test mit seinem Thermostat) und von @Tlse-vins (Test ohne kompatible Geräte + Tests mit meiner Instanz mit Thermostat). Abgesehen vom Anzeigefehler des Sollwerts (nicht mit diesem PR verbunden) sollte eigentlich alles in Ordnung sein. Läuft bei mir seit fast 3 Wochen.
Der Vorteil ist, dass es von jedem getestet werden kann, ich muss nur meine Instanz teilen. Wenn du Lust hast… sag mir Bescheid.
Nein ^^ Ich habe es getestet, aber es ist nichts möglich, feste Liste (vielleicht Rechte, aber ich glaube nicht…). Ich habe überall ein bisschen herumgesucht, aber nichts gefunden
Kleine Frage, die ich dir zunächst in der PR zur Netatmo-Integration gestellt habe, aber ich glaube, du hast nicht darauf geachtet:
Nach dem Release von v4.34.0 habe ich den Branch aktualisiert, aber wie üblich gab es einen Absturz im Codecov-Bericht. In deinem Live-Coding-Video hast du meine Frage beantwortet, wie man die Tests der PR neu starten kann:
Seltsam, ich habe es selbst neu gestartet, aber es ist wirklich nicht toll, wenn ein Entwickler eines Forks die Tests seines eigenen PRs nicht neu starten kann! ^^
Ich bin ein bisschen überrascht, die PR sieht überhaupt nicht mehr so aus, wie ich es gesehen hatte, als ich deinen Code nach dem Live-Coding angesehen hatte… Der Code ist meiner Meinung nach ziemlich weit entfernt von dem, was ich beim Live-Coding zeige!
Ich habe dir viele Punkte angegeben, die zu korrigieren sind. Nimm dir Zeit für die PR und versuche, sie vollständig zu lesen, um diese Änderungen überall anzubringen
In Bezug auf die Code-Menge sind wir wieder bei einer ziemlich fetten PR, also bin ich wirklich froh, dass wir uns auf den Thermostat beschränkt haben, das ist schon ein großes Stück zum Überprüfen
Vielen Dank für dein Review. Ich mache mich heute daran. Ich werde so viel Zeit darauf verwenden, wie nötig.
Es stimmt, aber findest du das normal, weil kein externes Modul wie bei Sonos verwendet wird, oder ist das für dich wirklich nicht normal? Denn wenn nicht, lohnt es sich vielleicht nicht, es zu korrigieren, sondern den gesamten Code komplett zu überarbeiten…
Beim überall verwendeten « netatmoHandler »… Ich nehme an, das ist ein falsches Verständnis/eine falsche Konfiguration meines VSCode. Das « this. » wird überall in meiner VSCode-Schnittstelle als ESLint-Fehler angezeigt, deshalb habe ich « netatmoHandler » nur aus diesem Grund übergeben.
Sind wir uns einig, dass es daran liegt, den gesamten Code zu kontrollieren und nicht über ein Node-Modul « netatmo » zu gehen? Oder ist das für dich schlecht geschrieben?
Ich stelle diese Fragen, weil ich sehr viel Zeit damit verbringe (mit Freude, versteht sich), aber wenn es dazu führt, dass du und ich viel Zeit verlieren, sollten wir nicht weitermachen. Wenn wir aber mit den Korrekturen, die du mir angibst, und der Zeit, die du darauf verwendest, « sicher » sind, dass wir es schaffen, dann gebe ich alles
Ich habe die gewünschten Änderungen vorgenommen (ich habe alles überprüft und deine Kommentare dreimal gelesen, ich hoffe, ich habe nichts vergessen, sonst bin ich wirklich schlecht) und die Tests aktualisiert, normalerweise ist alles korrigiert.
Wie ich dir bereits in der PR erwähnt habe, verstehe ich das Problem, das du hattest, nicht
Zitat Auf der funktionalen Seite
Wenn ich die Integration zum ersten Mal öffne, sehe ich Folgendes:
Es gibt sofort eine Fehlermeldung (ich habe nichts getan), was für den Benutzer etwas seltsam ist.
Ich habe versucht, es zu reproduzieren, aber ich kann nicht einmal deinen ersten Verbindungsfehler reproduzieren… Offensichtlich hatte niemand von den 2 Testern dieses Problem, und ich habe es gerade mit einer neuen Installation auf wsl wiederholt, ich habe dieses Problem nicht…, mein erster Verbindungsstatus:
Ich glaube, das war nur ein Problem mit der Verbindung zum lokalen Server während meiner Tests. Ich denke nicht, dass es mit dem PR zusammenhängt! Ich werde es noch einmal testen und dich auf dem Laufenden halten
Zögere nicht, wenn du Zugriff möchtest, für die Echtzeit-Ansicht der Geräte und das echte Testen, das kann besser sein, und das ist normalerweise nicht kompliziert! Ich habe gerade den Test mit einer anderen E-Mail-Adresse gemacht, es funktioniert gut, selbst ohne Netatmo-Geräte.
Da ich auf diesen PR warte, kann ich dann einen neuen PR basierend auf diesem ersten PR beginnen? Unter der Voraussetzung, dass es keine verlorene Arbeit ist, da es einfach ist, die Arbeit auch bei großen Änderungen während der neuen Überprüfung wiederherzustellen. Aber das ermöglicht mir, dranzubleiben und am Ende Zeit zu sparen.
Meine Frage bezieht sich vor allem darauf, welchen Einfluss das für dich hat? Ist es einschränkend oder problematisch, sobald der erste PR gemerged wurde?
Kein Problem, solange du die beiden PRs gut trennst
Entschuldige, im Moment bin ich mit dem Projekt beschäftigt, von dem ich im Live-Event zu Semesterbeginn gesprochen habe, daher hatte ich noch keine Zeit, die PRs zu reviewen!
Die Prioritäten ändern sich, und es ist unmöglich, sich zu verpflichten… Einfach eine Liste der Dinge, die du machen möchtest!
Hast du nicht die Berechtigungen, um diese Aktion durchzuführen?
