# PR-Review - Ich brauche euch! **URL:** \ **Category:** Développement\ **Created:** [18. Januar 2021 um 16:41 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030 "2021-01-18T16:41:56Z")\ **Posts on this page:** 14\ **Page:** 4
**Author:** ![Will9](https://gladys-assistant-community.b-cdn.net/letter_avatar_proxy/v4/letter/w/ebca7d/32.png) [@Will9](https://community.gladysassistant.com/u/Will9)\ **Post date:** [27. März 2021 um 06:36 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/61 "2021-03-27T06:36:32Z")
2 Projektoren und 2 Klimaanlagen ---
**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. März 2021 um 10:39 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/62 "2021-03-29T10:39:28Z")
> [@Terdious](#): > > Sag mal, wäre es nicht interessant, solche sehr interessanten Beiträge über Fortschritte sowie alle wichtigen Updates im „International“-Bereich des Forums zu verdoppeln, um diesen Bereich zum Leben zu erwecken, der, wie ich hoffe, immer wichtiger wird und damit sich die internationalen Mitglieder in das Projekt integriert fühlen? Klar 🙂 > [@Terdious](#): > > Alles top !! Ich hoffe, ich kann das am Wochenende testen !! Zögere nicht, mir Feedback zu geben, ich habe heute zwei neue Verbesserungen zu diesem Build hinzugefügt: - Der Merge von Zigbee2mqtt - Eine ziemlich wichtige Leistungsverbesserung von Gladys Plus 🙂 ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [6. April 2021 um 07:56 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/63 "2021-04-06T07:56:03Z")
@Will9 ich habe die Arbeit an Broadlink wieder aufgenommen, es gab eine Diskussion darüber, wie die Funktionen und die Anzeige zu verwalten sind. Ich werde also viele Dinge vereinfachen und verallgemeinern (und die Geräte „clim“ hinzufügen). Ansonsten für die anderen, **Aktueller Stand der PRs, die auf Überprüfung / Merge warten:** - Settings - Fehlermeldung: [Modification of the error handling for the name of the house on the front side by Terdious · Pull Request #1056 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1056) - ~~Zigbee2mqtt - Geräte: [https://github.com/GladysAssistant/Gladys/pull/1120~~](https://github.com/GladysAssistant/Gladys/pull/1120~~) - ~~Tasmota - Code bereinigen: [https://github.com/GladysAssistant/Gladys/pull/1122~~](https://github.com/GladysAssistant/Gladys/pull/1122~~) - ~~Szenen - Filter bei Auswahl: [https://github.com/GladysAssistant/Gladys/pull/1126~~](https://github.com/GladysAssistant/Gladys/pull/1126~~) - ~~Front - Warnung eslint: [https://github.com/GladysAssistant/Gladys/pull/1125~~](https://github.com/GladysAssistant/Gladys/pull/1125~~) - Szenen - Szenen mit Szenen: [Allows a scene to trigger another scene by rob-mccann · Pull Request #1124 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1124) - Withings: [https://github.com/GladysAssistant/Gladys/pull/1046](https://github.com/GladysAssistant/Gladys/pull/1046) **PRs, die auf Änderungen warten:** - CalDAV - Synchronisation: [https://github.com/GladysAssistant/Gladys/pull/1091](https://github.com/GladysAssistant/Gladys/pull/1091) (Kommentare) @bertrandda - eWelink: [Ewelink discover fix and enhancements by NickDub-old · Pull Request #1044 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1044) (Build fehlgeschlagen) @Pti_Nico - Dashboard-Diagramm: [Chart box on dashboard by euguuu · Pull Request #966 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/966) (Kommentare) @euguuu - ZWave-Dimmer: [zWave - Dimmer integration by sescandell · Pull Request #757 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/757) (Kommentare) @sescandell - Yeelight: [https://github.com/GladysAssistant/Gladys/pull/715](https://github.com/GladysAssistant/Gladys/pull/715) @Pti_Nico (du brauchst Hilfe, du scheinst festzustecken) - Szenen - Deaktivieren: [Disable scene by callemand · Pull Request #1075 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1075) (Kommentare) @Lokkye **Welcher ist der Status von:** - Hue Fade: [Hue fade in by cbellier · Pull Request #1041 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1041) @cbellier - Börse: [Stock exchange by ngeissel · Pull Request #1040 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1040) @ngeissel - Pushover: [Add New Service: Pushover by thebradleysanders · Pull Request #1014 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1014) @thebradleysanders - Arduino: [https://github.com/GladysAssistant/Gladys/pull/812](https://github.com/GladysAssistant/Gladys/pull/812) @billona Ich habe nur die PRs berücksichtigt, die nicht als Entwurf und nicht als WIP markiert sind. ---
**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:** [6. April 2021 um 09:25 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/64 "2021-04-06T09:25:05Z")
Wow, genial @AlexTrovato 🙂 Danke für die vollständige Rezension! ---
**Author:** ![Pti\_Nico](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pti_nico/32/273_2.png) [@Pti\_Nico](https://community.gladysassistant.com/u/Pti_Nico)\ **Post date:** [7. April 2021 um 12:21 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/65 "2021-04-07T12:21:41Z")
> [@AlexTrovato](#): > > - eWelink : [Ewelink discover fix and enhancements by NickDub · Pull Request #1044 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/1044) (build fail) @Pti_Nico Das ist ein Problem mit Sicherheitslücken in den globalen Abhängigkeiten, nicht im Zusammenhang mit dem Dienst: ```auto found 30 vulnerabilities (28 low, 2 high) in 1184 scanned packages run `npm audit fix` to fix 29 of them. 1 vulnerability requires manual review. See the full report for details. Error: Process completed with exit code 1. ``` Das sollte man für einen Fix auf Master überprüfen… > [@AlexTrovato](#): > > - Yeelight : [Yeelight service by NickDub · Pull Request #715 · GladysAssistant/Gladys · GitHub](https://github.com/GladysAssistant/Gladys/pull/715) @Pti_Nico (du brauchst Hilfe, du scheinst festzustecken) Ich wäre dankbar, wenn jemand mir helfen könnte, den Fehler korrekt zu behandeln, der von der Bibliothek nicht richtig abgefangen wird 😉 ---
**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:** [7. April 2021 um 13:16 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/66 "2021-04-07T13:16:07Z")
> [@Pti\_Nico](#): > > Das ist ein Problem mit globalen Abhängigkeiten, nicht mit dem Service selbst: Bist du sicher? Wir haben das jedenfalls nicht auf master, ich denke, wenn du master rebase, sollte das deinen Fehler beheben 🙂 ---
**Author:** ![Pti\_Nico](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pti_nico/32/273_2.png) [@Pti\_Nico](https://community.gladysassistant.com/u/Pti_Nico)\ **Post date:** [7. April 2021 um 17:12 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/67 "2021-04-07T17:12:55Z")
Tatsächlich behebt das Rebase das 😉 ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [9. April 2021 um 09:51 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/69 "2021-04-09T09:51:14Z")
@VonOx bereitest du etwa deine berufliche Umorientierung vor? Ich habe mir deine PR angesehen, das ist nicht schlecht!!! Aber ich habe ein paar Rückmeldungen. Ich denke, es wäre nett, wenn du den Docker-Befehl (docker-compose) bereitstellen könntest, um Pi-hole mit Ports zu starten, die nicht mit Gladys kollidieren. Das ist nicht kompliziert, aber auch nicht ganz trivial. ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [9. April 2021 um 10:23 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/70 "2021-04-09T10:23:10Z")
> [@AlexTrovato](#): > > Bereitest du wirklich deine berufliche Neuorientierung vor? Oh nein! 😅 Ich hätte damals im College „Scheiße“ sagen sollen, als man mir sagte, dass es im Softwarebereich keine Jobaussichten gäbe (ist wahr) und ich muss wirklich meine Präsentation aktualisieren, denn ich habe einen ziemlich besonderen beruflichen Werdegang. (Zuerst Zeichner/Konstrukteur, jetzt PLM-Berater/Integrator). Ich verheimliche nicht, dass mir das mehr zugesagt hätte, aber im Vergleich zu euch (den Entwicklern) scheint mir die Lücke unüberwindbar! * * * Aber zurück zu diesem PR. Wie du bemerkt hast, brauche ich eine Basis, Beispiele, um weiterzuentwickeln. Ich habe aus laufenden PRs und bestehenden Projekten geschöpft, um etwas zu erstellen, das einigermaßen funktioniert. Es gibt noch Dinge, die ich nicht beherrschen, und das merkt man, aber hey, man lernt jeden Tag dazu! Danke, dass du es getestet hast, und danke für das Feedback! ---
**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\ **Post date:** [9. April 2021 um 10:26 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/71 "2021-04-09T10:26:56Z")
Wenn du technische Antworten auf meine Anmerkungen möchtest, können wir uns per PN austauschen. Ich sage dir, was/wie du etwas machen sollst, warum man es so machen sollte und so weiter… 😉 Ich sage nicht nein zu einem neuen Dev 😃 ---
**Author:** ![Jean34](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/jean34/32/3349_2.png) [@Jean34](https://community.gladysassistant.com/u/Jean34)\ **Post date:** [10. April 2021 um 11:07 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/72 "2021-04-10T11:07:33Z")
Bist du dabei, den Linky-Dienst zu entwickeln? ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [10. April 2021 um 11:19 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/73 "2021-04-10T11:19:11Z")
Was bringt dich dazu, das zu sagen? ---
**Author:** ![Jean34](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/jean34/32/3349_2.png) [@Jean34](https://community.gladysassistant.com/u/Jean34)\ **Post date:** [10. April 2021 um 11:34 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/74 "2021-04-10T11:34:52Z")
> [@Linky-Zähler Verbrauchsdaten](https://community.gladysassistant.com/t/donnees-de-consommation-compteur-linky/6061/2?u=jean34): > > Hello Non car Gladys est en nodejs @Madmartigan a développé une gateway =\> [https://enedisgateway.tech](https://enedisgateway.tech) Pas besoin de lib spécifique , je suis en train de testé, a voir pour en faire une integration Gladys Das ist schon etwas älter, aber egal. Ich habe mich falsche Hoffnungen gemacht, lol. ---
**Author:** ![VonOx](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/vonox/32/10128_2.png) [@VonOx](https://community.gladysassistant.com/u/VonOx)\ **Post date:** [10. April 2021 um 12:06 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/75 "2021-04-10T12:06:52Z")
Um die Enedis-API zu nutzen, muss man eine juristische Person sein. Dies wird wahrscheinlich über Gladys Plus angeboten. [Vorherige Seite](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030.md?page=3)