Steckdose zum Blinken bringen

Bestellungen geliefert
1

Aufgrund der Entwicklung dieser Funktion für die Beleuchtung (und auf Anfrage, dass dasselbe für Steckdosen entwickelt wird), hier ein Testbild

docker pull cicoub13/gladys:blink-electric-plug

:warning: Dies ist ein Testbild, starten Sie es auf eigene Gefahr. Hier ist die Dokumentation für diese Art von Bild.

Die Entscheidung wurde getroffen, beide Typen (Lichter/Steckdosen) im selben Aktionstyp zu verwalten.

image
image1134×558 30.8 KB

@pierre-gilles Ich habe eine Migration entwickelt, um die bestehenden Szenen zu verwalten. Ich habe das Gefühl, dass alles richtig gemacht wird, und die Protokolle bestätigen dies ebenfalls, aber die Änderung wird nicht vorgenommen (als ob das Backup nicht funktioniert).

https://github.com/cicoub13/Gladys/blob/feat-blink-electric-plug/server/migrations/20240426100609-change-blinking-light-action.js

2024-04-28T21:49:46+0200 <info> 20240426100609-change-blinking-light-action.js:16 (Object.up) Scene migration: Found 1 scenes with light.blink action
2024-04-28T21:49:46+0200 <info> 20240426100609-change-blinking-light-action.js:21 () Scene migration: Updating action light.blink with device.blink
2024-04-28T21:49:46+0200 <info> 20240426100609-change-blinking-light-action.js:26 () Scene migration: Saving updated scene Blinking lights
2

cc @guim31 @GBoulvin

3

Was für eine Effizienz! Danke!
Ich bin unterwegs, ich kann es nicht vor Donnerstag testen :innocent:. Aber es gibt keinen Grund, warum es nicht klappen sollte :wink:

4

Das ist ja super :slight_smile: außer dass ich auch nicht testen kann, ich habe ein bisschen an meinem NAS rumgebastelt, also muss ich es erst wieder aufbauen, bevor ich meine Test-Gladys wieder einrichten kann…

5

Hallo @cicoub13, super effektiv :clap:

Kleine Frage, müssen wir die Aktion wirklich umbenennen?

Im Grunde genommen, selbst wenn wir den Benutzer eine Steckdose auswählen lassen, geht es doch darum, eine Lampe blinken zu lassen, oder?

Gibt es einen Fall, in dem ein Benutzer eine Steckdose blinken lassen möchte, um etwas anderes an der Steckdose zu verwenden? Das würde mich wundern

Für mich wird also nirgends von einer Steckdose gesprochen, selbst die Schnittstelle „Lichter blinken lassen“ ist ausreichend, nur dass der Benutzer eine Steckdose bei den Lichtern auswählen kann, oder?

6

Ah ja, wenn man etwas Abstand nimmt, ergibt das mehr Sinn. Und es erspart mir die Migration :wink: Ich ändere das

7

Das ist korrigiert. Bild cicoub13/gladys:blink-electric-plug bereit zum Testen.

PR hier Feat blink light from switch device by cicoub13 · Pull Request #2064 · GladysAssistant/Gladys · GitHub

8

@cicoub13 Ich habe es gerade getestet, bei mir funktioniert es gut!

Nur ein kleines UX-Feedback: Da die Listen der Lichter und Steckdosen verkettet sind, geht die Reihenfolge verloren und es kann schwierig sein, den Überblick zu behalten:

Screenshot 2024-05-01 at 13.52.01
Screenshot 2024-05-01 at 13.52.011242×904 41.9 KB

Vielleicht könnte man eine Sortierung zur Tabelle hinzufügen, damit alles gut sortiert ist?

Siehe Review: https://github.com/GladysAssistant/Gladys/pull/2064#pullrequestreview-2033501224

Danke für den PR auf jeden Fall :folded_hands:

9

Ja, ausgezeichnete Bemerkung.
Außerhalb davon läuft alles reibungslos!
Danke @cicoub13!

10

Änderung durchgeführt, PR und Docker-Image aktualisiert :white_check_mark:

11

Danke @cicoub13 für die Korrekturen, das ist für mich in Ordnung!

Es ist gemerged und wird in der nächsten Version von Gladys :rocket: veröffentlicht.

12

Diese Funktion ist in Gladys Assistant 4.40 verfügbar:

Ich schließe dieses Thema, zögern Sie nicht, ein neues zu erstellen, falls Feedback vorliegt :slight_smile:

Vielen Dank an @cicoub13 für die Entwicklung!