Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

removed the 'I do not understand you!' messages #221

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BartWillems
Copy link

When there are multiple bots in group chats, the octoprint bot keeps spamming I do not understand you! 😖.

example

@dentillozie
Copy link

thanks, this gets pretty annoying at times

@rlogiacco
Copy link
Collaborator

I don't like this solution: this means the bot will not provide any feedback in case you misspelt something and you don't know if the bot has received your message and decided not to interpret it or the message didn't reach your bot.

Can you provide this as a configurable option defaulting to sending the 'I don not understand you' message?

@BartWillems
Copy link
Author

Telegram auto completes commands, so I don't think that's an issue though.

@shreyasubale
Copy link

i so need this, was gonna do it myself. Thanks for saving me some time :)

@giloser
Copy link
Collaborator

giloser commented Feb 18, 2021

I will not accept this PR as some are not in group and doesn't dislike the bot to answer but I'll add an option to disable the answer in the next version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants