Use GET request if the media upload command is STATUS #425

Merged

ping merged 1 commit from mediaupload_status into master

2016-09-23 06:24:34 -07:00

ping commented

2016-06-22 07:27:31 -07:00

(Migrated from github.com)

The async media upload method requires an additional STATUS command, but unlike the other commands such as INIT, APPEND, FINALIZE, the STATUS command is a GET request. The patch switches the upload_media endpoint to use self.get() if the command param is 'STATUS'.

Twitter recently announced support for longer duration videos https://twittercommunity.com/t/api-support-for-140-second-video/69153 but to take advantage of that, the media must be uploaded via the async method. The async media upload method requires an additional STATUS command, but unlike the other commands such as INIT, APPEND, FINALIZE, the STATUS command is a GET request. The patch switches the upload_media endpoint to use self.get() if the command param is 'STATUS'.

coveralls commented

2016-06-22 07:29:40 -07:00

(Migrated from github.com)

Coverage decreased (-0.2%) to 56.094% when pulling f7ddbcf414 on ping:mediaupload_status into 2faa84629b on ryanmcgrath:master.

[![Coverage Status](https://coveralls.io/builds/6705876/badge)](https://coveralls.io/builds/6705876) Coverage decreased (-0.2%) to 56.094% when pulling **f7ddbcf414924ae31db4144bc5ffa16a000ca8ba on ping:mediaupload_status** into **2faa84629b3a42fc897bfa247cf20260cd527207 on ryanmcgrath:master**.

michaelhelmick (Migrated from github.com) approved these changes 2016-09-23 06:24:29 -07:00