Duplication de scènes

[mikael]

Bonjour,
Feature description

L’idée serait de pouvoir dupliquer une scène. ( utile quand on doit faire la même scène x fois )

Le plus serait lors de la copie de demander quel appareil doit être pris en compte.

exemple:

• Une scène initiale qui signale une batterie faible pour un appareil X.
• On duplique la scène et on précise que c’est pour un appareil Y ( attention je parle pas de personnaliser aussi le message envoyé si on utilise l’action « envoyer un message » )

[pierre-gilles]

Bonne idée pour la duplication !

@mikael

On duplique la scène et on précise que c’est pour un appareil Y ( attention je parle pas de personnaliser aussi le message envoyé si on utilise l’action “envoyer un message” )

La dessus je suis pas sûr, comment on fait si la scène est complexe et à des dizaines d’actions liées à plein de périphériques ?

Je pense que le plus simple c’est juste d’avoir un bouton dupliquer et ensuite de modifier la scène tout simplement

[mikael]

Descriptif très succinct :

• analyse de la scène à copier et récupération des valeurs des variables “appareils”
• une page qui affiche la valeur appareil par appareil de la scène copiée en premier dans une “select box”. Pour chaque “select box” le reste du choix sera du même type que l’appareil de la scène copiée.
• L’utilisateur pourra donc changer ses appareils
• Un bouton “sauvegarder” pour valider ses choix.

Pour cela il faut bien sûr que dans les propriétés des appareils que peut interroger gladys, que se trouve la propriété “type” d’appareil de renseigné et commune au même type d’appareil.

Je ne sais pas comment est codé une scène ( mode statique ( fichier texte brut ) ou dynamique ( via une base de données ) ) mais aussi comment sont référencés les appareils.

Je reconnais que çà peut faire une usine à gaz surtout si on est en mode statique.

[pierre-gilles]

Je suis pas convaincu, en gros tu propose de recoder la vue “édition de scène” dans une autre vue, je ne vois pas bien l’intérêt.

Autant juste copier la scène et ensuite l’utilisateur peut modifier la scène!

[mikael]

Ok c’était juste un plus, l’idée importante était la duplication de scène.

[Lokkye]

Bonjour, je viens de faire une petite PR pour rajouter la duplication d’un scène. Lorsque l’on click sur le bouton copier, cela envoie vers une nouvelle page pour choisir le nouveau nom de la scene dupliqué.

Duplicate scene

[Romuald_Pochet]

Bonjour,

Je viens de l’intégrer dans mon Gladys de PROD car j’y ai integré mes devices zwave et donc je dois créer les innombrables scenes pour gérer mes devices (pour toggle ON/OFF j’ai déjà besoin de 2 scenes et lorsque j’ai un interrupteur à 5 boutons . Ca “fonctionne” sauf que (et je ne pense pas que cela soit lié cette nouvelle fonctionnalité) mais lorsque l’on modifie un element par ex. l’appareil associé d’une action “Contrôler un appareil”, le champ valeur change automatiquement. De même, lorsque l’on modifie un trigger “Changement d’état de l’appareil”, la condition reste, la valeur reste également mais lors de la sauvegarde, j’ai une erreur car on envoie “null” dans la requete.

You save my life

[Lokkye]

Merci @Romuald_Pochet pour ton retour,
Je vais essayer de le reproduire de mon coté voir si je n’ai rien cassé. Est ce que ton problème apparait sur toutes les scenes ou seulement celle que tu as dupliqué ?

[Romuald_Pochet]

Non, cela se produit sur toutes les scenes

[Lokkye]

J’ai essayé de le reproduire mais n’ayant pas beaucoup de matériels connecté sur mon instance de dev ( je reset souvent la base des données pour avoir une instance propre). Je n’ai rien trouvé. Je te conseil plutôt de faire un bug sur le github

[Terdious]

Hi @Lokkye,

I just tested your PR, (struck-through = not tested) :

• On trigger :
  • Device state change
  • Scheduled trigger
  • At each sunrise
  • At each sunset
  • Return home
  • Leaving home
  • Home is empty
  • Home is no longer empty
  • User enters the zone
  • User leaves the zone
  • A calendar event occurs
• On action :
  • Turn on the light (keeping the same one and editing it),
  • Turn off the lights (keeping the same one and editing it),
  • Turn on the plugs (keeping the same one and editing it),
  • Turn off the plugs (keeping the same one and editing it),
  • Wait,
  • Send a message (keeping the same one and editing it),
  • Retrieve the last state (keeping the same one and editing it),
  • Continue only if (keeping the same one and editing it),
  • User seen at home (keeping the same one and editing it),
  • User left the home (keeping the same one and editing it),
  • Check presence (keeping the same one and editing it),
  • Make an HTTP request,
  • Time condition,
  • Start the scene (keeping the same one and editing it),
  • If the home is empty (keeping the same home and editing it),
  • If the home is not empty (keeping the same home and editing it),
  • Control a device (keeping the same one and editing it),
  • Condition on a calendar event,

I can’t reproduce @Romuald_Pochet’s case on « Control a device » (well, if you change the device the value does indeed fall back but that’s part of the base scene construction) but if I save immediately it works and if I put back the previous value (e.g. switch to On), likewise it saves correctly.

Note: If it’s possible to change the icon that would be perfect. Even though in most cases it won’t be the case, there are use cases where it can be useful ^^

Great PR ^^

[pierre-gilles]

For your information, I just reviewed the PR on GitHub => Duplicate scene by callemand · Pull Request #1505 · GladysAssistant/Gladys · GitHub

Overall, the feature works well, I just have a few UX comments! Nice work.

[pierre-gilles]

Quick follow-up on this topic — if anyone drops by here, development is no longer active and we are therefore looking for someone to take over development

The PR above was a good start, the work can be picked up again; you just need to address the various comments/feedback left on the PR.

[Lokkye]

Thank you very much for your feedback. I didn’t have as many test cases on my end. Great testing work.

[Lokkye]

I’m still here. I was just waiting for a few reviews. I’ll see about giving feedback ASAP.

[pierre-gilles]

@Lokkye

I’ll see about providing the feedback ASAP

Cool! We’ll probably need to do a small rebase too since quite a few things have changed since the PR!

Don’t hesitate to ask if you have any questions

[Lokkye]

@pierre-gilles: I’ve done the rebase and the changes on the PR but I can’t reopen it.
Could you check on your end if you can do it ?
Otherwise I’ll have to create a new one

[VonOx]

Are you sure you can’t ( by commenting)?

[pierre-gilles]

@Lokkye

@pierre-gilles: I’ve done the rebase and the changes on the PR but I can’t reopen it.
Could you check on your side to see if you can do it?

I can’t do it either!

You can recreate it, it’s not a problem

[Lokkye]

Here is the re-created PR: Duplicate scene by callemand · Pull Request #1670 · GladysAssistant/Gladys · GitHub