Fix upload_video "string argument detected, got 'bytes'" bug. #426

Merged
philgyford merged 1 commit from bugs/422 into master 2016-08-18 08:23:14 -07:00
philgyford commented 2016-08-17 06:21:57 -07:00 (Migrated from github.com)

Needed to use BytesIO instead of StringIO when uploading video to Twitter.

Tested with python 2.6.9, 2.7.11, 3.3.6, 3.5.1.

fixes #422

Needed to use BytesIO instead of StringIO when uploading video to Twitter. Tested with python 2.6.9, 2.7.11, 3.3.6, 3.5.1. fixes #422
coveralls commented 2016-08-17 06:24:02 -07:00 (Migrated from github.com)

Coverage Status

Coverage decreased (-0.2%) to 56.065% when pulling 1b20f8f0f9 on philgyford:bugs/422 into 2faa84629b on ryanmcgrath:master.

[![Coverage Status](https://coveralls.io/builds/7487118/badge)](https://coveralls.io/builds/7487118) Coverage decreased (-0.2%) to 56.065% when pulling **1b20f8f0f9eeeb398c89feb1787e541b21098741 on philgyford:bugs/422** into **2faa84629b3a42fc897bfa247cf20260cd527207 on ryanmcgrath:master**.
coveralls commented 2016-08-17 06:24:02 -07:00 (Migrated from github.com)

Coverage Status

Coverage decreased (-0.2%) to 56.065% when pulling 1b20f8f0f9 on philgyford:bugs/422 into 2faa84629b on ryanmcgrath:master.

[![Coverage Status](https://coveralls.io/builds/7487118/badge)](https://coveralls.io/builds/7487118) Coverage decreased (-0.2%) to 56.065% when pulling **1b20f8f0f9eeeb398c89feb1787e541b21098741 on philgyford:bugs/422** into **2faa84629b3a42fc897bfa247cf20260cd527207 on ryanmcgrath:master**.
coveralls commented 2016-08-17 06:24:02 -07:00 (Migrated from github.com)

Coverage Status

Coverage decreased (-0.2%) to 56.065% when pulling 1b20f8f0f9 on philgyford:bugs/422 into 2faa84629b on ryanmcgrath:master.

[![Coverage Status](https://coveralls.io/builds/7487118/badge)](https://coveralls.io/builds/7487118) Coverage decreased (-0.2%) to 56.065% when pulling **1b20f8f0f9eeeb398c89feb1787e541b21098741 on philgyford:bugs/422** into **2faa84629b3a42fc897bfa247cf20260cd527207 on ryanmcgrath:master**.
philgyford commented 2016-08-17 06:24:51 -07:00 (Migrated from github.com)

Dammit; decreased coverage due to removing lines! :)

Dammit; decreased coverage due to removing lines! :)
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#426
No description provided.