PR-Review - Ich brauche euch!

Entwicklung
61

2 Projektoren und 2 Klimaanlagen

62

Klar :slight_smile:

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 :slight_smile:
63

@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:

PRs, die auf Änderungen warten:

Welcher ist der Status von:

Ich habe nur die PRs berücksichtigt, die nicht als Entwurf und nicht als WIP markiert sind.

64

Wow, genial @AlexTrovato :slight_smile: Danke für die vollständige Rezension!

65

Das ist ein Problem mit Sicherheitslücken in den globalen Abhängigkeiten, nicht im Zusammenhang mit dem Dienst:

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…

Ich wäre dankbar, wenn jemand mir helfen könnte, den Fehler korrekt zu behandeln, der von der Bibliothek nicht richtig abgefangen wird :wink:

66

Bist du sicher? Wir haben das jedenfalls nicht auf master, ich denke, wenn du master rebase, sollte das deinen Fehler beheben :slight_smile:

67

Tatsächlich behebt das Rebase das :wink:

69

@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.

70

Oh nein! :sweat_smile: 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!

71

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… :wink:
Ich sage nicht nein zu einem neuen Dev :smiley:

72

Bist du dabei, den Linky-Dienst zu entwickeln?

73

Was bringt dich dazu, das zu sagen?

74

Das ist schon etwas älter, aber egal. Ich habe mich falsche Hoffnungen gemacht, lol.

75

Um die Enedis-API zu nutzen, muss man eine juristische Person sein. Dies wird wahrscheinlich über Gladys Plus angeboten.