# Netatmo Service **URL:** \ **Category:** Développement\ **Tags:** netatmo\ **Created:** [8. Mai 2020 um 17:54 UTC](https://community.gladysassistant.com/t/service-netatmo/5910 "2020-05-08T17:54:28Z")\ **Posts on this page:** 20\ **Page:** 15
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [24. Januar 2024 um 09:29 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/282 "2024-01-24T09:29:14Z")
Ok, super, danke für die Antwort. Kann ich dich bitten, kurz einen Blick auf den neuen PR zu werfen: [[CANCELED #2014] - Netatmo add valves NRV by Terdious · Pull Request #2005 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2005), um zu bestätigen, dass alles in Ordnung ist? Ich möchte sicher sein, dass die Methodik stimmt, wenn man einen PR von einem anderen Branch aus erstellt und welche Auswirkungen die Commit-Historie hat, wobei in diesem neuen PR die Historie des anderen PRs zu sehen ist. Und dass bei der Erstellung des neuen PRs die Tests des vorherigen erneut gestartet wurden… immer ein bisschen beunruhigend, wenn man das nicht oft macht ^^ Vielen Dank im Voraus! ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [26. Januar 2024 um 08:12 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/283 "2024-01-26T08:12:14Z")
> [@Terdious](#): > > Kannst du mir bestätigen, dass alles in Ordnung ist? Ich glaube, du hast dich vertan, denn ich sehe genau die gleiche Anzahl von Commits in einem PR wie im anderen, und mit genau denselben Commits. ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [26. Januar 2024 um 11:57 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/284 "2024-01-26T11:57:54Z")
Hallo @pierre-gilles! Ja also, wenn man es so macht wie ich, ist das normal, ich habe bisher nichts gepusht, ich habe lokal gearbeitet. Genau deshalb meine Frage. Ich bin von meinem Basis-Branch netatmo-integration ausgegangen, um meinen neuen Branch basierend auf origin/gladys zu erstellen (um mit dem Code dieses ersten PRs zu beginnen und mit der Integration fortzufahren). Ich habe nur den neuen Branch gepusht und dann den neuen PR erstellt, der mit dem GladysAssistant-Repo verknüpft ist. Ist das die richtige Methode, um sauber zu bleiben? ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [26. Januar 2024 um 13:19 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/285 "2024-01-26T13:19:38Z")
Ah okay, in dem Fall ist alles gut 🙂 Vergiss nur nicht, in den richtigen Branch zu committen. ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [26. Januar 2024 um 18:38 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/286 "2024-01-26T18:38:25Z")
Super, danke für die Bestätigung ^^ Ja, bei den Pushes bin ich nach all dem Unsinn, den ich mit meinem persönlichen Projekt gemacht habe, jetzt besser 😅 ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [28. Januar 2024 um 14:19 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/287 "2024-01-28T14:19:04Z")
Neuer PR bereit, der die NRV-Ventile hinzufügt: > > > \### Pull Request check-list > > To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items: > > \- \[x\] If your changes affects code, did your write the tests? > \- \[x\] Are tests passing? (\`npm test\` on both front/server) > \- \[x\] Is the linter passing? (\`npm run eslint\` on both front/server) > \- \[x\] Did you run prettier? (\`npm run prettier\` on both front/server) > \- \[x\] If you are adding a new features/services, did you run integration comparator? (\`npm run compare-translations\` on front) > \- \[x\] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community (\[french forum\](https://community.gladysassistant.com/)/\[english forum\](https://en-community.gladysassistant.com/)) for testing before merging. > > NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open. > > \### Description of change > > Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some! > > Cancelled and replaced by #2014 Testimage wird gebaut: `docker pull terdious/gladys:netatmo-add-valves` ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/6/63e79ddc2fcda91f3fd9230a2b3bcf2d59da6d16.png) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/2/237e80d80d9bf21ca805725824e2cc900f113983.png) Ich mache weiter mit der Wetterstation. ---
**Author:** ![Will\_71](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/will_71/32/9582_2.png) [@Will\_71](https://community.gladysassistant.com/u/Will_71)\ **Post date:** [28. Januar 2024 um 16:26 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/288 "2024-01-28T16:26:08Z")
Man kann dich einfach nicht mehr stoppen… ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [29. Januar 2024 um 06:54 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/289 "2024-01-29T06:54:11Z")
Neue PR bereit, die die Innenwetterstationen hinzufügt: [[WIP] Add support for NAMain module type - Weather / Smart Home Weather Station #2010](https://github.com/GladysAssistant/Gladys/pull/2010) Testbild wird gebaut: `docker pull terdious/gladys:netatmo-add-weather-station` ![image](https://gladys-assistant-community.b-cdn.net/images/transparent.png) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/a/a7703c76972b7af9ac59634beda70ac875a80ad1.png) Ich werde mit den Modulen der Wetterstation eines nach dem anderen weitermachen. > [@Will\_71](#): > > Man kann dich nicht aufhalten… 😅 in der Hoffnung, dass die erste PR gut genug ist ^^ aber normalerweise habe ich die Zweige angehängt, damit die Aktualisierungen vereinfacht werden. Das sollte die anderen PRs nicht zu sehr beeinflussen, falls Änderungen in der Überprüfung verlangt werden!! Edit: > [@PhilippeMA](#): > > Hallo, > > Ich bin bereit, bei den Tests zu helfen. > > Ich habe eine vollständige Wetterstation (inklusive Regenmesser und Anemometer). > > Danke, dass du dich um die Entwicklung dieser Funktion gekümmert hast 🙂 @PhilippeMA, ich informiere dich, wenn ich mit der vollständigen Station fertig bin, damit du testen kannst, wenn es immer noch in Ordnung ist! ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [29. Januar 2024 um 12:04 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/291 "2024-01-29T12:04:47Z")
Neue PRs bereit, die hinzufügen: - die Innenmodule der Wetterstationen: [[CANCELED #2016] PR n°3 : Add support for NAModule4 type - Weather / Smart Indoor Module by Terdious · Pull Request #2011 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2011) - dann die Außenmodule der Wetterstationen: [[CANCELED #2017] PR n°4 : Add support for NAModule1 type - Weather / Smart Outdoor Module by Terdious · Pull Request #2012 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2012) Test-Image im Bau, das alle laufenden Netatmo-PRs integriert: `docker pull terdious/gladys:netatmo-add-module-outdoor-weather-station` ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/8/820a480e2ee6dd496d3efeb1d266d42289593016.png) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/c/c00c950a790685cd81050b687c55777b588d46c7.png) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/0/09e6149f273ccfbb4345e0f8026dec95dc23e2e2.png) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/7/7ff798159566c35ed804d3ae168939ec941e568d.png) ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [29. Januar 2024 um 12:16 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/292 "2024-01-29T12:16:59Z")
@Terdious Review auf dem ersten PR gemacht! Es ist viel besser 🙂 Ich habe ein paar Feedback-Punkte, aber nichts Kompliziertes! > > > \### Pull Request check-list > > To ensure your Pull Request can be accepted as fa…st as possible, make sure to review and check all of these items: > > \- \[x\] If your changes affects code, did your write the tests? > \- \[x\] Are tests passing? (\`npm test\` on both front/server) > \- \[x\] Is the linter passing? (\`npm run eslint\` on both front/server) > \- \[x\] Did you run prettier? (\`npm run prettier\` on both front/server) > \- \[x\] If you are adding a new features/services, did you run integration comparator? (\`npm run compare-translations\` on front) > \- \[x\] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community (\[french forum\](https://community.gladysassistant.com/)/\[english forum\](https://en-community.gladysassistant.com/)) for testing before merging. > \- \[x\] If you are adding a new features/services which needs explanation, did you modify the user documentation? See \[the GitHub repo\](https://github.com/GladysAssistant/v4-website) and the \[website\](https://gladysassistant.com). > > NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open. > > \### Description of change > > Add a new Netatmo integration > OAuth2 authentication management ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [29. Januar 2024 um 14:19 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/293 "2024-01-29T14:19:28Z")
Vielen Dank @pierre-gilles! Die Korrekturen sind erledigt!! Ich übernehme sie in die anderen PRs. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [29. Januar 2024 um 14:57 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/294 "2024-01-29T14:57:04Z")
Danke für das Feedback! Für den Wettbewerb würde ich empfehlen, bei maximal 2 zu bleiben, für Dinge, die so im Hintergrund laufen. Es gibt Benutzer auf Pi, wir wollen eine leichte Software bleiben und große Spitzen vermeiden 🙂 Halte mich auf dem Laufenden, sobald die Dokumentation aktualisiert ist. Teste alles noch einmal gründlich, um sicherzustellen, dass du nichts kaputt gemacht hast, als du die Korrekturen vorgenommen hast, und für mich können wir dann die erste Version in die Produktion bringen 🙂 ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [29. Januar 2024 um 20:45 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/295 "2024-01-29T20:45:10Z")
Alles klar für mich @pierre-gilles, - Wettbewerb auf 2 festgelegt: ✅ - Neue vollständige Tests auf neuer Installation nach den Fixes der Überprüfung: ✅ - Dokumentation aktualisiert mit den aktuellen Bildern (en und fr): ✅ - Hinzufügen einer Dashboard-Ansicht zum Abschluss: ✅ - Docker-Image `terdious/gladys:netatmo-integration-dev` bei Bedarf aktualisiert: ✅ Edit: Die folgenden 4 PRs sind zur Überprüfung bereit mit den Aktualisierungen der ersten. Du kannst dich damit beschäftigen, sobald du Zeit hast, aber nur zur Info, sie sind wirklich sehr leicht, wie du sehen wirst. Bisher habe ich nur das hinzugefügt, was bereits existiert (außer bei der ersten der Wetterstation, die 1 oder 2 zusätzliche Konstanten erforderte, aber nichts Schlimmes). - Docker-Image `terdious/gladys:netatmo-add-module-outdoor-weather-station` für diejenigen, die es wünschen: ✅ - Regenmesser (einfach) und Anemometer (etwas schwieriger - Front-Dashboard-Hinzufügung) in Arbeit: ✴ ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [30. Januar 2024 um 05:29 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/296 "2024-01-30T05:29:45Z")
Natürlich musste Codecov beim letzten Push abstürzen (ich kann sie immer noch nicht neu starten), aber die Tests laufen lokal gut, keine Sorge: ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/5/54f88f19e8d8086c840e8977f6ca87c5c867ff06.png) Edit: Es ist hässlich, aber mein einziger Weg, sie erneut auszuführen, ist ein doppelter „Fake Push“… Bei mir tritt das Problem eindeutig auf, sobald mindestens 2 PRs in den Tests sind. Gestern ist das dreimal passiert, als ich in kurzer Zeit Pushes auf verschiedene PRs gemacht habe. ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [30. Januar 2024 um 07:28 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/297 "2024-01-30T07:28:25Z")
> [@Terdious](#): > > C’est tout bon pour moi @pierre-gilles, Super 😎 Danke für deine Schnelligkeit!! Für mich ist es okay, es ist auf Github akzeptiert und in master gemerged 🚀 Die Dokumentation ist sehr sauber! Ich werde die Dokumentation zum Zeitpunkt des Releases mergen (wahrscheinlich nächste Woche) > [@Terdious](#): > > Edit: Okay, das ist nicht schön, aber mein einziger Weg, sie durchzubekommen, ist ein doppelter „Fake Push“… du hast recht 😃 Es ist kein Problem, wenn es schmutzige Commits gibt, da es nur in der PR ist, ich werde danach squash mergen. ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [30. Januar 2024 um 23:20 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/298 "2024-01-30T23:20:17Z")
> [@Terdious](#): > > Edit: Die folgenden 4 PRs sind bereit zur Überprüfung mit den Aktualisierungen der ersten. Du kannst dich darum kümmern, sobald du Zeit hast, aber nur zur Info: Sie sind wirklich sehr leicht, wie du sehen wirst. Bisher habe ich nur das hinzugefügt, was bereits existiert (außer bei der ersten der Wetterstation, die 1 oder 2 zusätzliche Konstanten benötigte, aber nichts Schlimmes). > > - Docker-Image `terdious/gladys:netatmo-add-module-outdoor-weather-station` für diejenigen, die es wünschen: ✅ > - Regenmesser (einfach) und Anemometer (etwas schwieriger - Front-Dashboard hinzufügen) in Arbeit: ✴ Letztendlich bin ich etwas zurückgegangen, habe die Zweige und PRs vom aktuellen Master aus neu erstellt. Es war zu umständlich, sie zu aktualisieren, und jetzt ist es viel sauberer. Ich habe also 2 PRs zur Überprüfung bereit: - Die NRV-Ventile: [PR n°1 : Add support for NRV module type - Energy / Valves by Terdious · Pull Request #2014 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2014) - Der Hauptwetterstationssensor für den Innenbereich: [[MERGED #2021] PR n°2 : Add support for NAMain module type - Weather / Smart Home Weather Station by Terdious · Pull Request #2015 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2015) Vollständig getestet: ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/1/1442a3daa7baa670411c652023dd975fe0906374.png) ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [31. Januar 2024 um 18:00 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/299 "2024-01-31T18:00:58Z")
Fortschritte bei der Wetterstation: ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/b/b8833e0cdd0280573207b4104ce2f816907586ba.png) Regenmesser und wir sind bereit, um weiterzumachen! ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [1. Februar 2024 um 18:04 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/300 "2024-02-01T18:04:40Z")
Alles klar für die Wetterstation: ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/b/b46bc4c10112200cc24ee1ba258f92e9dd39deb4.png) @pierre-gilles, kannst du die 6 PRs überprüfen (ich habe sie unabhängig gemacht, aber die Reihenfolge ist zumindest für PR 1 und 2 wichtig). Und du sagst mir, ob du es vorziehst, ich habe einen Zweig, der alle 6 Geräte integriert. Das kann die kleinen Korrekturen, die nach und nach vorgenommen werden, erleichtern und vor allem die Überprüfung (46 Dateien im einzigen Zweig gegen 6 PRs mit jeweils etwa 20 Dateien, da die meisten Dateien in jeder PR betroffen sind). @PhilippeMA, wenn du motiviert bist, die gesamte Wetterstation zu testen: `docker pull terdious/gladys:netatmo-features-nrv-weather` ---
**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\ **Post date:** [2. Februar 2024 um 13:53 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/301 "2024-02-02T13:53:57Z")
> [@Terdious](#): > > du wirst die 6 PRs (ich habe sie unabhängig gemacht, aber die Reihenfolge ist zumindest für die PRs 1 und 2 wichtig) überprüfen können. Okay, super 🙂 Ich denke, ich werde am Montag ein Release machen, ich mache bereits die erste Version. Dann überprüfe ich den Rest und das wird in der nächsten Version veröffentlicht. Klingt das gut? Danke für deine Reaktionsschnelligkeit, es ist wirklich toll, all diese PRs zu sehen!! ---
**Author:** ![Terdious](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/terdious/32/6316_2.png) [@Terdious](https://community.gladysassistant.com/u/Terdious)\ **Post date:** [2. Februar 2024 um 14:10 UTC](https://community.gladysassistant.com/t/service-netatmo/5910/302 "2024-02-02T14:10:13Z")
Super!! Das klingt sehr gut für mich ^^ Ich werde den PR mit den Punkten 1 bis 6 hinzufügen ([Grouping of PRs 2 to 6: Addition of Weather functionality by Terdious · Pull Request #2021 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/2021)). So kannst du sehen, was am interessantesten zum Reviewen/Mergen ist. Es ist ein Vergnügen!! Danke für dein Feedback! [Vorherige Seite](https://community.gladysassistant.com/t/service-netatmo/5910.md?page=14) [Nächste Seite](https://community.gladysassistant.com/t/service-netatmo/5910.md?page=16)