Pouvoir envoyer une image de caméra par message dans les scènes

Créer une nouvelle action dans les scènes « envoyer une image de caméra » qui laisse sélectionner la caméra + l’utilisateur a qui envoyer l’image

J’avais un peu de temps libre ce matin donc j’ai commencé à travailler sur ce sujet.

J’ai fait une PR (qui est en Draft pour le moment)

L’action fonctionne, il me reste juste a faire les tests unitaires :slight_smile:

5 « J'aime »

Super ça :ok_hand:, on ne t’arrête plus !!

1 « J'aime »

Voila le resultat:


Screenshot 2023-05-31 at 20.24.46

7 « J'aime »

Il te fait récupérer un changement d’état d’un capteur pour ça !

Hyper propre :heart_eyes: merci pour la PR, tiens moi au courant quand c’est prêt pour être review !

1 « J'aime »

@pierre-gilles: Normallement, la PR est prête

1 « J'aime »

Je viens de faire une review:

Je viens de corriger les retours :slight_smile:

1 « J'aime »

Trop cool merci pour les retours rapides! J’ai testé, et ça marche nickel chez moi :slight_smile:

J’ai juste une petite remarque sur la taille du titre de l’action, ça rend pas super:

Il faudrait trouver un titre plus petit (mais quand même clair), une idée ?

Je pense qu’on peut enlever « par message » non ?

2 « J'aime »

Voici les propositions de ChatGPT :

  • « Envoyer une photo par message »
  • « Envoyer une capture de caméra par message »
  • « Envoyer une capture caméra par message »
  • « Envoyer une capture caméra en message »
  • « Envoyer capture caméra par message »
  • « Envoyer capture caméra par msg »
  • « Envoyer capture cam par msg »
  • « Transmettre une image caméra par message »
  • « Envoyer une capture caméra en message »
  • « Partager une photo caméra par message »
  • « Transférer une image caméra par message »
  • « Envoyer une capture vidéo par message »
  • « Transmettre une photo de caméra par message »
  • « Partager une image de caméra par message »
  • « Envoyer une prise de vue caméra par message »

Voila Voila :sweat_smile:

Je suis d’accord avec @guim31, « envoyer une image de caméra » c’est suffisant non ?

Oui, je pense aussi

Je viens de pousser les changements sur la PR

3 « J'aime »

Effectivement, c’est bien mieux :

Pour moi la PR est bonne, le code est bon, et j’ai testé chez moi ça fonctionne parfaitement :slight_smile:

Beau boulot, c’est mergé :clap:

4 « J'aime »

Cette fonctionnalité est disponible dans Gladys Assistant 4.25, merci à @Lokkye pour le développement :pray:

Je ferme ce sujet, pour d’éventuels bugfix je vous invite à créer un nouveau sujet :pray:

2 « J'aime »