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

[ad-690] Add shutdown notice #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

alexpryshchepa
Copy link

No description provided.

Copy link
Collaborator

@wilsonpage wilsonpage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC this will message the group/user everytime a music link is shared, I feel like this might be quite irritating. Is there a way we can just message once and remove the bot from the group? Or perhaps we just prompt them to remove the bot.

IMO the path of least resistance would be to just stealth close the bot and not make any announcement. People probably won't notice for a while that the bot isn't functioning.

@alexpryshchepa
Copy link
Author

IIUC this will message the group/user everytime a music link is shared, I feel like this might be quite irritating. Is there a way we can just message once and remove the bot from the group? Or perhaps we just prompt them to remove the bot.

IMO the path of least resistance would be to just stealth close the bot and not make any announcement. People probably won't notice for a while that the bot isn't functioning.

Correct, we will show the message on every page creation. Don't know how often people use telegram bot and it was the simplest option to make awareness of future shutdown. When the message shown they can remove it from the group by their own I think. Might be frustrating to silently shutdown the bot. We can't send the message to all users as we do not store chat ids, but I agree that we can store the flags for particular chats in memory to determine if message shown.

btw, we do need to change the words as we agreed to shutdown public version completely.

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