-
Notifications
You must be signed in to change notification settings - Fork 117
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
Telegram bot command to filter favorites #93
base: master
Are you sure you want to change the base?
Conversation
I have to try this when I get the time. I would like this way better if the favorites would be stored per chat, instead of globally. This would make it possible to share the bot with family / friends that live in the same town:
|
is there any news on this feature of sharing it so that multiple users can have separate lists of favourites? And so that they can login with their own account and manage their favourites using their own account? or a more broad approach of instanced user sessions?
|
Easiest way to achieve multi-tenancy would actually be simulating it. Because of API request limitations per IP it would not be viable sending requests for multiple accounts with one bot. Simple approach would be making the user able to add / remove favorites by chat, for example using vendor or bag ids. The chat would subscribe to receiving notifications for the bags it added / subscribe to already existing bags. Those bags would be added to one dummy account, especially created and authenticated for the bot, and when the bot detects changes it would send notifications to each subscribed chat. Implementing this would probably have to be supported by starting to use some kind of database to store subscriptions though, for example a local sqlite. |
mmh that sounds pretty doable and a good upgrade indeed, thanks for the info/suggestion. Will try this out after exam period. Having all people receive all notifications in the telegram bot chat and having to 'subscribe' via one account , is quite annoying but not that necessary a fix either. Let me know though @marklagendijk if you ended up doing something like this already , so I don't end up doing something redundant. |
I haven't done anything like this. Main reason being that I don't have a real need for it. The approach sounds fine. |
Try an implementation of this feature request #60