Skip to content
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

Video upload #62

Closed
YOzaz opened this issue Feb 19, 2015 · 13 comments
Closed

Video upload #62

YOzaz opened this issue Feb 19, 2015 · 13 comments

Comments

@YOzaz
Copy link

YOzaz commented Feb 19, 2015

Marking it here to keep it on track - Twitter is going to add video upload support soon, so I would love this library to be updated as well (for Laravel 4).

https://twittercommunity.com/t/twitter-video-support-in-rest-and-streaming-api/31258/15

@beeseenmedia
Copy link

or laravel 5 :)

@YOzaz
Copy link
Author

YOzaz commented Aug 25, 2015

Any update for this?
I think we need chunked upload possibility, described here: https://dev.twitter.com/rest/reference/post/media/upload-chunked

@beeseenmedia
Copy link

i edited the code to add video support. you can see it here. Note there is a second variable to the function which specified if media is a video

beeseenmedia@d875ea6

@YOzaz
Copy link
Author

YOzaz commented Aug 26, 2015

@beeseenmedia thanks for reply. In general, I think second parameter can be omited by detecting MIME type form media source.

@beeseenmedia
Copy link

indeed. it can. I just wanted a quick and dirty way to do it but yes a mime detection would make it neater, feel free to fork, modify and commit.

@YOzaz
Copy link
Author

YOzaz commented Sep 24, 2015

Pull request is here: #105.
@thujohn any chance you'll merge this soon?

@YOzaz
Copy link
Author

YOzaz commented Apr 6, 2016

Updated pull request to avoid memory issues: #136

@scottybo
Copy link

Just in case someone else finds this thread, I've forked this package and added video support (docs for video near the bottom of the readme): https://github.com/scottybo/twitter

@FrankChinedu
Copy link

@scottybo i tried running the upload video but i get RuntimeException (400)

@atymic
Copy link
Owner

atymic commented Mar 21, 2021

@reliq is this supported in 3.x?

@reliq
Copy link
Collaborator

reliq commented Mar 21, 2021

@reliq is this supported in 3.x?

Unfortunately, I'm not able to confirm this actually works any better OTB in v3, uploadMedia works as it did in v2.

If this continues to be an issue for some people I could invest some more time into it.

@atymic
Copy link
Owner

atymic commented Apr 13, 2021

This can be closed now right @reliq?

@reliq
Copy link
Collaborator

reliq commented Apr 13, 2021

Yes. This is fixed @atymic

@reliq reliq closed this as completed Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants