Bibo-Joshi · GitHub

@BAcode-X

Copy link Copy Markdown

Contributor

Based on Telegram API doc, the parameter can only be up to 64 characters long.

Checklist

- [ ] Added new constants at `telegram.constants`

@BAcode-X

@Bibo-Joshi

@Poolitzer

Copy link Copy Markdown

Member

Hi. You added the constant above the line where it says that "the constants above this line are tested", while not adding a test. I don't think this variable needs one (and it wouldn't be fun to write either), so just move it down.

@BAcode-X

Poolitzer

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

LGTM

@Bibo-Joshi

@Bibo-Joshi

Copy link Copy Markdown

Member

Thank you for the contribution :)

Read the original on github.com ↗