Point PR Januar 2023

Hallo an alle @contributors! :folded_hands:

Eine kleine Nachricht, um den Stand der laufenden PRs für 2023 zu besprechen :slight_smile:

Ich glaube, ich habe alle PRs durchgesehen. Gibt es noch PRs/Issues, die auf eine Rückmeldung warten?

Ich versuche mein Bestes, aber Github ist auf der Seite der Benachrichtigungen nicht besonders gut gemacht. 90% der Benachrichtigungen, die ich erhalte, sind nutzlos, daher nutze ich sie nicht wirklich (ich erhalte eine Benachrichtigung bei jedem Commit in einer PR… super nützlich, und ich habe es nie geschafft, das zu deaktivieren).

Ich habe dieses Update korrigiert

EDIT: Ich wusste nicht, dass die Migrationsdateien getestet werden …

Dieses hier ist auch fertig, aber es gibt kein Feedback von den Nutzern

Die Tests scheinen aber nicht durchzulaufen?

Ja, die Migrationsläufe funktionieren in den Tests, sonst wäre die DB nicht da :smiley:

Ok, kannst du es vielleicht im Forum nochmal posten?

das ist logisch, aber ich sehe nicht, wie ich das Problem lösen soll:

20230102171731-update-tasmota-energy-features.js:26:33
  1) "before all" hook: before in "{root}"

  0 passing (6s)
  1 failing

  1) "before all" hook: before in "{root}":
     TypeError: Cannot read properties of null (reading 'id')
      at Object.up (migrations/20230102171731-update-tasmota-energy-features.js:26:33)

Ich verstehe den Fehler: Die Tabelle t_service in der Test-DB ist leer. Ich habe einen Seeder erstellt, aber das ändert nichts.

Was ist also die richtige Methode? Soll man das catchen und die Migrationsdatei stoppen?

Warum nicht, aber man muss diese Migration trotzdem testen, sonst schickt man eine ungetestete Migration in die Wildnis und das ist kritisch, denn wenn die Migration abstürzt, startet Gladys nicht mehr.

Der PR tasmota ist in Ordnung @pierre-gilles

Ich starte den anderen neu

Guten Abend,

Was den PR Zwave betrifft, habe ich bereits Feedback von Mathieu erhalten, der bereits ein Problem angemerkt hat, das ich behoben habe.
Da ich quasi der einzige Nutzer von ZWave bin (42 ZWave-Knoten und 72 Gladys-Geräte), ist der einzige Punkt, der wirklich in der Praxis getestet werden muss, die Einrichtung der Docker-Container, selbst wenn wir keine ZWave-Geräte haben. Die 42 Knoten sind funktionsfähig (was nicht für alle meine Zigbee-Geräte gilt).

Zwavejs2mqtt by rpochet · Pull Request #1619 · GladysAssistant/Gladys · GitHub

Kann man sich mit einem bestehenden Container verbinden?

Ja, indem Sie die Option „Externe ZwaveJS UI“ auswählen

Ich habe nur ein Rollladenmodul, ich teste das heute Abend.

@pierre-gilles kannst du die PR von alex aktualisieren, um einen neuen Build zu erstellen?

Danke

Das ist für diejenigen, die vor kurzem auf die Zwave JS UI migrieren und zurückwechseln mussten, wirklich eine sehr sehr gute Sache!

Hallo zusammen!

Ich bin gut in Asien angekommen und werde morgen voll durchstarten mit der Entwicklung :slight_smile:

Ich habe gesehen, dass es einige PRs gibt, die noch ausstehen, die nehme ich mir morgen alle vor.

Wenn es eurer Meinung nach dringende Themen gibt, zögert nicht, mir das hier zu sagen, und ich werde sie mir ansehen.

Im Moment werde ich mich für 4 intensive Wochen auf Gladys konzentrieren.

Meine Ziele sind:

  • Alle Verbesserungen, die seit Weihnachten entwickelt wurden, ab morgen veröffentlichen (Szenenduplikation, etc. Alles, was auf master ist)
  • Eine erste Version der Enedis-Linky-Integration in Produktion für die aktuellen Gladys-Plus-Nutzer, in „privater Alpha“-Version
  • Alle ausstehenden PRs überprüfen
  • Bei Gladys Plus: Wiederherstellung des Jahresplans, neue Zahlungsmethoden hinzufügen (ich denke an Bancocontact für unsere belgischen Freunde, verfügbar bei Stripe, aber ich habe es nie aktiviert), über verschiedene Angebote nachdenken: unterschiedliche Preise? Zwei-Jahres-Pläne wie bei NordVPN zum Beispiel? Ein Plan, der ein „Starter-Paket“ mit Hardware von unserem Partner Domadoo beinhaltet? Da muss ich noch nachdenken.
  • Verbesserung der Entwicklerdokumentation
  • Große UX-Überarbeitung für alles bei Gladys: einen oder zwei Tage Entwicklung ausschließlich für UX widmen, um alle kleinen Anfragen umzusetzen, die zwar simpel sind, aber irgendwann erledigt werden müssen: Dashboard neu anordnen, Szenen neu anordnen, etc.
  • Überlegen, wie ich den Umsatz von Gladys Plus nutzen kann, um mehr Wachstum für das Projekt zu erzielen. Wettbewerb? Partnerschaft? Einen Dienstleister bezahlen (wenn der Umsatz es zulässt)?

Das ist ambitioniert, aber sehr motivierend!

Während dieser 4 Wochen, je schneller ihr reagiert, desto mehr kann ich schaffen, also zögert nicht!

Erledigt! Das Build wird in ein paar Minuten fertig sein :+1: Entschuldigt für die Wartezeit, aber für das wäre es am besten, wenn @AlexTrovato Cloudflare Pages auf seinem Konto einrichtet (es ist kostenlos), dann hast du die Builds direkt und musst nicht warten, bis ich ein „offizielles“ Build erstelle.

Das ist großartig! Gut gesehen :sunglasses:

Das hat gereicht, um einen schönen Tag zu haben :blush::blush::smiling_face_with_sunglasses:

Genau so :heart:
Danke @pierre-gilles :blush:

@VonOx hast du eine Idee, warum die „Legacy-Tags“ im letzten Build der Gladys-Version nicht funktionieren?

https://github.com/GladysAssistant/Gladys/actions/runs/3926693292/jobs/6712720389

Aus einem mir unbekannten Grund gibt es kein Manifest.

Ich reproduziere es lokal

❯ docker manifest inspect gladysassistant/gladys:latest | jq -r '.manifests | to_entries[] | select(.value.platform.architecture == "arm" and .value.platform.variant == "v6").value | .digest'
no such manifest: docker.io/gladysassistant/gladys:latest
❯ docker manifest inspect gladysassistant/gladys:latest
no such manifest: docker.io/gladysassistant/gladys:latest
❯ docker manifest inspect gladysassistant/gladys:v4
no such manifest: docker.io/gladysassistant/gladys:v4
❯ docker manifest inspect gladysassistant/gladys
no such manifest: docker.io/gladysassistant/gladys:latest

Ich setze die Untersuchung fort

Ich reproduziere das auf meinem Test-Repo.

Ich versuche, buildx herunterzustufen (v0.9.1), das ist die letzte Version, die einen erfolgreichen Build durchgeführt hat (die neueste Version ist v10.0.0).

EDIT: Es funktioniert, ich mache einen PR, um die Releases freizugeben, und ich werde die Issues im Docker-Repo überwachen.

Toll, danke @VonOx!

Ich habe heute Morgen einige Merges gemacht :sunglasses: Danke für eure PRs!