# 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:** 20\ **Page:** 3
**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:** [22. März 2021 um 11:14 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/41 "2021-03-22T11:14:48Z")
Gute Nacht dann 😁 ---
**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:** [23. März 2021 um 01:00 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/42 "2021-03-23T01:00:35Z")
Abgezeichnet für den Sonnenaufgang! ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/5/519ad6b93340964d8459a249ff4ea38cc3bfaf71.jpeg) Und bei euch? ---
**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:** [23. März 2021 um 06:49 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/43 "2021-03-23T06:49:47Z")
Ja, bei mir ist alles gut. ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [23. März 2021 um 08:19 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/44 "2021-03-23T08:19:40Z")
Hallo, Ich würde gerne bei den Tests der neuen PRs in Gladys helfen: - Gibt es ein dediziertes Docker-Image, in dem man einen bestimmten Branch klonen kann? - Hat jemand eine Anleitung geschrieben? - Gibt es Infos, die ihr mir weitergeben könnt? Danke für eure Antworten 😃 ---
**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:** [23. März 2021 um 09:14 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/45 "2021-03-23T09:14:42Z")
Auf welchem OS bist du? Man muss nicht unbedingt ein Docker-Image bauen, um es zu testen ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [23. März 2021 um 09:25 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/46 "2021-03-23T09:25:46Z")
Raspberry Pi + Docker. **[edit]** Ich dachte daran, einen zweiten Container auf dem Pi laufen zu lassen, mit einem anderen Port und einer anderen Datenbank, um die neuen Vorschläge zu bewerten. Daher meine Frage, ob es bereits ein solches Image gibt, mit Gladys und Git, um zwischen den Branches zu wechseln. ---
**Author:** ![Tlse-vins](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/tlse-vins/32/6054_2.png) [@Tlse-vins](https://community.gladysassistant.com/u/Tlse-vins)\ **Post date:** [23. März 2021 um 10:51 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/47 "2021-03-23T10:51:06Z")
Ich glaube, er möchte wissen, ob du Windows oder Mac nutzt. ---
**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:** [23. März 2021 um 11:03 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/48 "2021-03-23T11:03:29Z")
Hast du den PC neben dir? Ich habe den Edit nicht gesehen, es gibt einen Dev-Tag, aber das ist, was in den Master-Branch gemerged wurde und noch nicht released ist (die validierten PRs). Wir erstellen kein Bild für jede PR. Wenn du also eine PR testen möchtest, musst du das manuell tun oder ein Bild bauen. ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [23. März 2021 um 11:11 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/49 "2021-03-23T11:11:11Z")
Im schlimmsten Fall werde ich das Bild mit dem Tag dev verwenden, Git darauf installieren und den Branch klonen, der mich interessiert. ---
**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:** [23. März 2021 um 11:12 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/50 "2021-03-23T11:12:48Z")
Das hat überhaupt keinen Sinn, du wirst in einem Container manuell arbeiten. ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [23. März 2021 um 11:53 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/51 "2021-03-23T11:53:30Z")
Ja, ich bin mir dessen bewusst, aber es sind drei Befehle (install, pull und start). ---
**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:** [23. März 2021 um 12:09 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/52 "2021-03-23T12:09:46Z")
Da irrst du dich Okay, du ziehst den Code und erstellst den Container mit dem dev-Image (das ist einfach). In diesem Image musst du: - Gladys stoppen - Gladys-Installation löschen - Systemabhängigkeiten installieren - Repo klonen - Richtigen Branch auschecken - npm-Pakete installieren - Frontend bauen und in den Ordner server kopieren - Gladys starten Außerdem, wenn du Watchtower hast, sobald ein neues dev-Image gebaut wird, verlierst du deinen Container, weil er überschrieben wird. Du hast die Dokumentation für die Entwicklungsumgebungen =\> [Mettre en place un environnement de développement sous Mac/Linux | Gladys Assistant](https://gladysassistant.com/fr/docs/dev/setup-development-environment-mac-linux) (Windows / Mac und Linux) ---
**Author:** ![lmilcent](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/lmilcent/32/7023_2.png) [@lmilcent](https://community.gladysassistant.com/u/lmilcent)\ **Post date:** [23. März 2021 um 12:58 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/53 "2021-03-23T12:58:25Z")
Danke für deine Zusammenfassung, das hilft mir, nichts zu vergessen. Ich kenne Docker gut und war mir aller Aspekte bewusst, die du angesprochen hast. Aber ich kenne die Gladys-Entwicklungsumgebungen noch relativ wenig (obwohl ich bereits viel NodeJS entwickelt habe). Nach diesen Diskussionen werde ich mit dem [Dockerfile](https://github.com/GladysAssistant/Gladys/blob/master/docker/Dockerfile) von Gladys beginnen und es ändern, um mein Dev-Image basierend auf einem bestimmten Branch und auf Anfrage zu _builden_. Das wird einfacher sein, um einen „Haus“-Dev-Container zu erstellen, basierend auf einem bestimmten PR, wenn ich ihn testen möchte / kann. ---
**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:** [23. März 2021 um 14:01 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/54 "2021-03-23T14:01:20Z")
Der GitHub CI ist für den Build eines speziellen Zweigs vorgesehen (auf Anfrage) ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/c/c9b4a917aaa92d8bb127f4c2b952f5e42a55d12b.png) Du musst nur deine Geheimnisse in deinem Fork definieren ![image](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/b/baa32efdb6943565cd1f97d083b8da73564a520f.png) Auf dem Raspberry wird es zu lange dauern Die Dockerfile ist für docker buildx im Mehrstufenverfahren vorgesehen. ---
**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. März 2021 um 02:22 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/55 "2021-03-26T02:22:07Z")
Hallo zusammen! Ich denke, es wäre cool, alle PRs zu schließen, die nicht mehr aktiv sind. Es wird langsam schwierig, die Liste der PRs zu durchsuchen, und für neue Leute sieht es aus wie ein „verlassenes“ Repo. Es wirkt, als wäre es nicht gut gepflegt. Was denkt ihr dazu? 🙂 Falls ihr PRs habt, die nicht kurz- oder mittelfristig gemerged werden sollen, könntet ihr bitte etwas aufräumen? Danke euch allen 🙏 ---
**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. März 2021 um 09:40 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/56 "2021-03-26T09:40:13Z")
Zur Info, ich habe ein Build `gladysassistant/gladys:dev` mit allen kürzlich gemergten neuen Funktionen erstellt, und es gibt viele davon, insbesondere: - Node 14 - Sequelize 6 - Sonnenaufgang/Sonnenuntergang in Szenen - Helligkeitsauswahl im Dashboard - Node-nlp neueste Version und neue Satzgruppen - Feuchtigkeitsbox - Aktualisierung der Abhängigkeiten - Aktualisierung der Philips Hue-Bibliothek & neue lokale Suche - und vieles mehr! Alle aktuellen Änderungen: > **[Comparing v4.1.1...master · GladysAssistant/Gladys](https://github.com/GladysAssistant/Gladys/compare/v4.1.1...master)** > > A privacy-first, open-source home assistant. Contribute to GladysAssistant/Gladys development by creating an account on GitHub. Ich lade euch ein, es zu testen und mir zu sagen, ob bei euch alles weiter funktioniert ^^ ---
**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. März 2021 um 10:00 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/57 "2021-03-26T10:00:53Z")
Hallo @pierre-gilles, Super, alles das!! Ich hoffe, ich kann das dieses Wochenende testen!! Sag mal, wäre es nicht interessant, solche sehr interessanten Beiträge über die Fortschritte sowie alle wichtigen Updates im „International“-Bereich des Forums zu doppeln, damit dieser Bereich, der ich hoffe, immer wichtiger wird, lebendig bleibt und sich die internationalen Mitglieder in das Projekt eingebunden fühlen? ---
**Author:** ![cicoub13](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/cicoub13/32/4482_2.png) [@cicoub13](https://community.gladysassistant.com/u/cicoub13)\ **Post date:** [26. März 2021 um 10:43 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/58 "2021-03-26T10:43:03Z")
> [@pierre-gilles](#): > > Falls ihr PRs habt, die nicht kurz- oder mittelfristig gemerged werden sollen, könntet ihr da ein bisschen aufräumen? Kannst du ein Label „Veraltet“ oder „Verlassen“ erstellen, damit wir sie markieren können? Einige PRs wurden von Leuten erstellt, die nicht mehr am Projekt mitarbeiten ---
**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:** [26. März 2021 um 12:25 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/59 "2021-03-26T12:25:28Z")
Kann dieser PR integriert werden? > > > \### 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) > \- ~~If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)~~ > \- ~~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/gladys-4-docs) and the \[website\](https://documentation.gladysassistant.com).~~ > \- \[x\] Did you add fake requests data for the demo mode (\`front/src/config/demo.json\`) so that the demo website is working without a backend? (if needed) See \[https://demo.gladysassistant.com\](https://demo.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 > > Handle Broadlink WiFi IR devices and smart plugs. Ich hatte eine Testversion von Gladys mit diesem integrierten Dienst und es funktionierte bei mir gut. ---
**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:** [26. März 2021 um 19:34 UTC](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030/60 "2021-03-26T19:34:17Z")
Ich werde das Thema noch einmal aufgreifen müssen. Mit welchen Geräten arbeitest du? TV? Licht? [Vorherige Seite](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030.md?page=2) [Nächste Seite](https://community.gladysassistant.com/t/review-des-pr-jai-besoin-de-vous/6030.md?page=4)