Skip to content

Bot is usable in groups now#15

Merged
smartnode merged 2 commits intosmartnode:masterfrom
jakob-stark:master
Nov 3, 2018
Merged

Bot is usable in groups now#15
smartnode merged 2 commits intosmartnode:masterfrom
jakob-stark:master

Conversation

@jakob-stark
Copy link
Contributor

There have been two issues with the code concerning group chat IDs.

  • Chat IDs may be up to 56bit long but most variables holding chat IDs were 32bit integers
  • Telegram group chat IDs may be negative. In the telebot library there were many checks to prevent sending to chats with negative ID and throwing an invalid argument error.

… Thats a bug because in Telegram all group chat_id values are negative
@smartnode smartnode merged commit 26b060b into smartnode:master Nov 3, 2018
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.

2 participants