| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
@Bibo-Joshi Last time I faced this error in the support groups I suggested checking the dimensions of it. Maybe try uploading said photo as document, if that works, that might be it? |
Sorry, something went wrong.
|
@Poolitzer Problem seems to be sending a photo via with open('tests/data/thumb.jpg', 'rb') as f:
bot.send_photo(chat_id, f)
Problem persists, if I swat tests/data/telegram.jpg with telegram.png or thumb.jpg. However, I have no problems sending the to myself with a personal bot(token). Could that be a type of flood control? We have been getting some flooding errors in the tests recently … @tsnoam maybe we can add new/replace the existing tests bots? |
Sorry, something went wrong.
There was a problem hiding this comment.
great PR, slight improvement
Sorry, something went wrong.
|
One thing: What about adding the emojis from Dice as constants? We normally do that. Right now we only have a list. |
Sorry, something went wrong.
Added that. Still kept the ALL_EMOJI list, though |
Sorry, something went wrong.
|
@Bibo-Joshi Sure, didnt complain about that ;P |
Sorry, something went wrong.
|
CI Fail: test_max_caption_length gives an Internal server error for some reason. seems unrelated. |
Sorry, something went wrong.
There was a problem hiding this comment.
In general looks good. A small comment on docstring.
However, I did not understand the following action item in the PR description:
Release notification should maybe include a statemant, that Dice a second positional argument.
Sorry, something went wrong.
Co-authored-by: Noam Meltzer <tsnoam@gmail.com>
| Back | FazBrowse Home | New Git URL |
Pure API changes
Convenience stuff
Todo