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

Allow multiple chat_id for Telegram storage. #123

Open
Gurulhu opened this issue Jun 1, 2022 · 1 comment
Open

Allow multiple chat_id for Telegram storage. #123

Gurulhu opened this issue Jun 1, 2022 · 1 comment

Comments

@Gurulhu
Copy link
Contributor

Gurulhu commented Jun 1, 2022

It may be a corner case and I understand this can be "solved" by putting the bot in a shared chat, but I bumped into the issue of wanting to notify more than one user on Telegram, and as it is now the implementation only supports one chat id.

As a side note, the telegramstorage.py file is missing a PastieStorage import that sometimes silently breaks the storage class if no other storage class that imports it is enabled and loaded before it.

I fixed both issues in my fork and will open a PR if you deem the multiple chat_ids issue relevant.
If you chose to deny it, I can rollback the fork and PR only to fix the import issue, just let me know.

@cvandeplas
Copy link
Owner

Such contribution would be nice. Thank you

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

No branches or pull requests

2 participants