X Tutup
Skip to content

Add ability for custom filename in send methods and InputFile#57

Merged
leandrotoledo merged 6 commits intopython-telegram-bot:testingfrom
ergoz:testing
Sep 8, 2015
Merged

Add ability for custom filename in send methods and InputFile#57
leandrotoledo merged 6 commits intopython-telegram-bot:testingfrom
ergoz:testing

Conversation

@ergoz
Copy link
Copy Markdown
Contributor

@ergoz ergoz commented Sep 8, 2015

Methods sendAudio, sendVideo, sendVoice, sendDocument and other that uses real files updated to add ability to set custom filename that will show in telegram client.

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.

and self.data['filename'] is redundant of if 'filename' in self.data, so you can keep only the first part.

@leandrotoledo
Copy link
Copy Markdown
Member

Nicely done, please just make the minor changes I've noted and we'll be good to merge in.
Thank you for this PR!

leandrotoledo added a commit that referenced this pull request Sep 8, 2015
Add ability for custom filename in send methods and InputFile
@leandrotoledo leandrotoledo merged commit ed77afa into python-telegram-bot:testing Sep 8, 2015
@leandrotoledo
Copy link
Copy Markdown
Member

Yay! Thanks. I'll take care of these Travis tests errors, it wasn't your PR but some TelegramError parse issues.

@ergoz
Copy link
Copy Markdown
Contributor Author

ergoz commented Sep 8, 2015

Telegram api server 502 bad gateway :) How to run travis building again?

@leandrotoledo
Copy link
Copy Markdown
Member

That's a good question! haha
Refresh doesn't seem to be working. I'll be sending tests shortly though. But I've been pulling this change into my workspace to run tests locally, so far so good.

@github-actions github-actions bot locked and limited conversation to collaborators Aug 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

X Tutup