Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

[Feature Request] Notifier Interaction #1550

Closed
LoneWolf345 opened this issue Apr 13, 2018 · 1 comment
Closed

[Feature Request] Notifier Interaction #1550

LoneWolf345 opened this issue Apr 13, 2018 · 1 comment

Comments

@LoneWolf345
Copy link
Contributor

Describe the Feature

I would like the ability to "Talk" to zenbot through a connected notifier. If, for example, I have discord configured with zenbot I would like to send the message "What are my current balances for Binance" and have it respond with all my current holdings. Also have things like switching to manual mode so that it will stop trading. There are many other examples as you can imagine. But the first step is to create that framework.

@tuxitor
Copy link
Contributor

tuxitor commented Apr 14, 2018

@LoneWolf345
What you ask for and a lot more are actually done in another fork, but a lot of water have flooded into the ocean since then. You may look at this to get an idea:

https://github.com/tuxitor/zenbot/blob/zentalk/docs/Zentalk.md

"The Zentalk Concept" actually worked pretty well at that time, except maybe for auto websockeet address:port on systems other than Linux This work was done before any serious notifier system was implemented in Zenbot, but now a lot of those notifiers are implemented in a fashion that ZenTalk doesn't fit into.

If someone is interested in following up on this, "The Zentalk Concept" is still living in the zentalk branch here: https://github.com/tuxitor/zenbot/tree/zentalk

emabo added a commit to emabo/zenbot that referenced this issue Apr 10, 2020
…mands also from a connected notifier.

Implemented only for Telegram at the moment.
 - list of accepted commands is the same used in command line
 - check the correct origin of the message looking at chat ID
 - accept the flag 'c.notifiers.telegram.interactive' (default = false)
   to enable/disable this feature

This commit provides a possible solution for feature request DeviaVir#1550
YarnSeemannsgarn pushed a commit to YarnSeemannsgarn/zenbot that referenced this issue Apr 15, 2020
…1550 (DeviaVir#2070)

* stale[bot] (DeviaVir#1744)

* fix: upgrade lint-staged from 10.1.1 to 10.1.2 (DeviaVir#2063)

Snyk has created this PR to upgrade lint-staged from 10.1.1 to 10.1.2.

See this package in NPM:
https://www.npmjs.com/package/lint-staged

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade ccxt from 1.25.80 to 1.25.81 (DeviaVir#2064)

Snyk has created this PR to upgrade ccxt from 1.25.80 to 1.25.81.

See this package in NPM:
https://www.npmjs.com/package/ccxt

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* Exchanges: update-products 5838

* fix: upgrade css-loader from 3.4.2 to 3.5.0 (DeviaVir#2067)

Snyk has created this PR to upgrade css-loader from 3.4.2 to 3.5.0.

See this package in NPM:
https://www.npmjs.com/package/css-loader

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade ccxt from 1.25.81 to 1.25.83 (DeviaVir#2066)

Snyk has created this PR to upgrade ccxt from 1.25.81 to 1.25.83.

See this package in NPM:
https://www.npmjs.com/package/ccxt

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade semver from 7.1.3 to 7.2.1 (DeviaVir#2065)

Snyk has created this PR to upgrade semver from 7.1.3 to 7.2.1.

See this package in NPM:
https://www.npmjs.com/package/semver

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* Exchanges: update-products 5852

* fix: upgrade css-loader from 3.5.0 to 3.5.1 (DeviaVir#2069)

Snyk has created this PR to upgrade css-loader from 3.5.0 to 3.5.1.

See this package in NPM:
https://www.npmjs.com/package/css-loader

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* fix: upgrade ccxt from 1.25.83 to 1.25.86 (DeviaVir#2068)

Snyk has created this PR to upgrade ccxt from 1.25.83 to 1.25.86.

See this package in NPM:
https://www.npmjs.com/package/ccxt

See this project in Snyk:
https://app.snyk.io/org/deviavir/project/14e19887-e219-40d4-89b6-6c657bf78942?utm_source=github&utm_medium=upgrade-pr

* Exchanges: update-products 5862

* Notifier interaction: add events to notifiers in order to receive commands also from a connected notifier.

Implemented only for Telegram at the moment.
 - list of accepted commands is the same used in command line
 - check the correct origin of the message looking at chat ID
 - accept the flag 'c.notifiers.telegram.interactive' (default = false)
   to enable/disable this feature

This commit provides a possible solution for feature request DeviaVir#1550

Co-authored-by: Chase <chase@sillevis.net>
Co-authored-by: Snyk bot <snyk-bot@snyk.io>
Co-authored-by: Travis CI <travis@travis-ci.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants