Ajout d'une option "user" à la commande "Tous" #31
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Allows to target a specific user via a variable/tag
Allows to target a specific chatId via a variable/tag
|
Petit ajout: j'ai ajouté l'option chatId qui fonctionne de la même manière. Cela permet de jouer directement avec l'information "Expéditeur" sans la traiter pour récupérer le nom. |
|
Cela dit je viens de m'apercevoir que ça ne règle pas totalement le problème puisque dans la commande Ask il est demandé la commande d'envoi sans possibilité de spécifier des options... |
|
Je vais intégrer le PR, pourrais tu ajouter un bout de doc sur la fonction en PR sur la documentation ici : https://github.com/lunarok/jeedom_docs/blob/master/plugins/telegram/index.md |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bonjour @lunarok ,
Merci pour ce plugin qui fonctionne très bien, cependant je suis confronté à un problème et comme je pense ne pas être le seul je fais cette pull request: envoyer un message à un destinataire récupéré dans une variable/tag.
Dans mon cas d'utilisation, j'ai des scénarios qui exécutent plusieurs commandes Ask via Telegram les unes à la suite des autres (exemple: "Ouvrir les volets ?", "Allumer la radio ?", etc).
Ne sachant qui va répondre, j'envoie évidemment sur "Tous". Une fois que quelqu'un à répondu, il est possible de savoir à qui envoyer les autres questions puisqu'il suffit de récupérer cette information via la commande "Expéditeur", sauf qu'il est impossible dans l'éditeur de scénario de Jeedom d'utiliser cette information pour cibler le destinataire.
J'ai donc ajouté une option "user" dans la commande "Tous" qui permet de cibler un utilisateur en particulier.
Cette option prend en paramètre le nom de la commande liée à l'utilisateur. Evidemment on peut y ajouter les autres options habituelles (comme parse).
Comme des images valent mieux que mille mots, voici des captures du fonctionnement:
Je pense que cette fonction serait certainement très utile. Je n'ai pas trouvé d'autre moyen de la mettre en place mais je pense cependant que cette solution n'est pas mauvaise et est facile à utiliser.
Merci de bien vouloir jeter un œil dessus (ça ne représente qu'une dizaine de lignes de codes) :)