Da es sich um eine für mich essentielle Funktion handelt und ich Gladys überhaupt nicht beherrschen, habe ich mir erlaubt, einen PR für diese Anfrage zu erstellen (https://github.com/GladysAssistant/Gladys/pull/1338).
@Romuald_Pochet du hast den falschen Branch gewählt, da der PR deine Änderungen an der zwave betrifft oder du hast deinen Branch nicht von master aus erstellt
PS: Entschuldigung, aber ich habe mich noch nicht daran gewöhnt, Git zu verwenden… Aber wenn es einfacher ist, kann ich von der Master-Branch neu beginnen.
Ok, ich erstelle einen sauberen Branch. Also, ich kann nicht von Gladys master ausgehen, oder? Sollte ich nicht einen Branch in meinem Fork erstellen? Oder mein master enthält auch zwavejs, ich glaube, das ist das Problem, ich hätte einen speziellen Branch für meine Demos erstellen sollen…
Bevor wir über den Code sprechen, denke ich, es wäre gut, wenn wir uns darauf einigen, was wir auf Feature-Ebene wollen, denn für mich gibt es viele Möglichkeiten, das zu tun, und es ist kein einfaches Feature:
Typischerweise ist der Trigger „Ereignis startet“ nützlich, aber oft möchte man „Ereignis startet in 5/15/30 Minuten“ (wenn du z. B. eine Erinnerung 15 Minuten vor einem Termin erhalten möchtest). Da es sich um ein völlig anderes Paradigma handelt, wird dies die Implementierung enorm ändern (da jeder Trigger und nicht jedes Ereignis programmiert werden muss), also sollte man vorher darüber nachdenken.
Wie funktioniert die Auswahl des Ereignisses? Gibt es eine Liste und du wählst ein Ereignis aus der Liste aus? Nimmst du alle Ereignisse, die diese Zeichenkette enthalten? Meiner Meinung nach muss man sich hier wirklich Gedanken machen, es kann nützlich sein, einen Trigger zu haben, der bei einem bestimmten Ereignis oder bei einer bestimmten Textübereinstimmung ausgelöst wird, in jedem Fall muss es in der UI klar sein.
Entschuldige, wenn ich viele Fragen stelle, aber es ist ein wirklich cooles Feature und wenn wir den Bedürfnissen aller gerecht werden wollen, müssen wir uns vorher die Fragen stellen!
Danke auf jeden Fall, dass du mit der Entwicklung begonnen hast @Romuald_Pochet, es ist auf jeden Fall eine schöne Arbeit
Ja, die Funktion ist tatsächlich noch unvollständig! Ich habe gerade erst mit der Entwicklung begonnen, was auch in den Kern von Gladys (und vor allem den UI-Teil) eingeht;-) Ist das nicht eine gute Arbeitsweise?
Der Reminder wird über den Reminder von CalDav verwaltet. Aber das Datenmodell „calendar_event“ müsste aktualisiert werden, um den Wert einzuschließen; Man könnte auch den Reminder bei der Erstellung der Szene angeben und somit den Auslöser bei der Erstellung der Szene erstellen. In diesem Fall geht jedoch die Auslösung außerhalb der Szene verloren, z. B. eine Benachrichtigung in der UI oder über Telegram.
„Informationen für Eventbedingungen abrufen“: zu erledigen
Die aktuelle Bedingung ‹ user › + ‹ Kalender › + ‹ Event ›, wobei ‹ Event › ein regulärer Ausdruck ist (z. B. Bedingung Papa/Mama funktioniert für Papa, Mama Papa/Mama, ich gebe zu, das ist ziemlich spezifisch für meinen Bedarf)
Die Rechte werden nicht verwaltet und das scheint mir tatsächlich ziemlich komplex zu sein.
Kein Problem, es gab nichts Negatives in meiner Nachricht !!
Genau das wird diskutiert, meiner Meinung nach ist das eher etwas, das wir in der Szene als im Kalender verwalten möchten, denn typischerweise könnte man mehrere Szenen wollen, die auf dasselbe Event ausgelöst werden, zum Beispiel:
30 Minuten vor dem Termin, mir eine Telegram-Benachrichtigung senden
15 Minuten vorher, wenn ich das Haus nicht verlassen habe, einen Ton im Haus abspielen
5 Minuten vorher, alle Lichter im Wohnzimmer blinken lassen!
Meiner Meinung nach sollte die Programmierung im Trigger und nicht im Kalender spezifiziert werden, es ist meiner Meinung nach auch einfacher in Bezug auf die Benutzerfreundlichkeit.
Dazu sollte man vielleicht mehrere Fälle vorschlagen:
Auswahl eines bestimmten Ereignisses (ist das nützlich?)
Ereignis, dessen Titel diesen Text enthält (und dazu muss man in der UI klar sein, das Wort « regex » ist nicht für jeden offensichtlich )
Ich denke, das Gespräch mit @bertrandda sollte mit dieser Entwicklung im Hinterkopf weitergehen, damit wir etwas haben, das im Ganzen kohärent ist
Das ist der Sinn
Jedenfalls herzlichen Glückwunsch zur ersten Entwicklung, das ist toll! Meine Anmerkungen sind nur positiv gemeint, das Ziel ist es vor allem, voranzukommen!
Ich korrigiere das so schnell wie möglich
