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

Unlimited message size #997

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

Conversation

Daniel-97
Copy link

@Daniel-97 Daniel-97 commented Aug 18, 2022

  • eslint tests pass
  • I have run npm run doc

Description

This PR try to solve the issue with the error "Message too long" when sending message larger than 4096 characters.
I have added an additional function sendLargeMessage() for sending message larger than 4096 char.
The max message size right now is 4096 since there is no specification on the official telegram documentation of the maximum message size for premium account (i assume the limit is the same as a non-premium account).

This PR is a better implementation of the closed PR #992 since:

  • This is an additional functionality and will not break existing code
  • The function don't use async/await so it doesn't break current code style.

References

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.

None yet

1 participant