-
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
docs: fix links #614
docs: fix links #614
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great changes but please revert the unrelated whitespace changes to the source code
Line 2366 in 8ebf31c
Here the anchors are outdated and can be updated to video-requirements and animation-requirements. Should I include those as well? Since those docs are from the Bot API docs itself, I didn't know what to do. |
My bad. Fixing them... |
Are they also outdated in the official Telegram docs? |
Yes |
|
Then we are going to keep broken links in our documentation, as well, until Telegram fixes the issue. This is going to happen soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much!
@dcdunkan are you planning on opening similar PQs to the other repositories in the organisation? |
Yes, in my free time. |
We can fix it now, it was already done in grammyjs/types@2987b30 for the types |
No description provided.