Add support for videos
See original GitHub issueReact-static-tweets
doesn’t current support videos. Here’s an example tweet:
On Twitter: https://twitter.com/sassal0x/status/1364207094614749184 Rendered Version: https://react-static-tweets.vercel.app/1364207094614749184
Would love any pointers on how/where one might start to implement this! 🙂
Issue Analytics
- State:
- Created 2 years ago
- Reactions:1
- Comments:9 (2 by maintainers)
Top Results From Across the Web
Five ways to add video into your customer support strategy
Five ways to add video into your customer support strategy · 1. Use pre-made videos to answer repetitive questions · 2. Resolve support...
Read more >Strava Adding Video Support: Here's how it works!
Strava has begun rolling out video support, allowing you to upload short video clips to your workouts, which will appear in your feed...
Read more >Add subtitles and captions - YouTube Help - Google Support
Play the video and enter your captions or subtitles. Don't forget to add text like [applause] or [thunder] , so that viewers know...
Read more >How to Add Videos - SNO Sites Support
You need to use a third-party video hosting service to store and serve video. These services are built specifically for serving high quality ......
Read more >Adding Video to Pages and Posts - INN Technical Support
For other video hosting providers, you may need to add some or all of these manually. In any case, you can edit them...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
@kevcodez I got it working but as mentioned in my last comment, it’d be better imo to do a more global refactor of the AST fetching code, because my current solution duplicates the calls to Twitter’s API and I think there might be a way to avoid it.
Maybe we could ship this under an option so that it’s opt-in for users who don’t mind the cost (extra requests could lead to problems with Twitter’s API rate-limits). Sort of like an experimental feature flag. What do you think @transitive-bullshit?
@kevcodez if you need to use this right now, feel free to build the package from my branch yourself, but be aware of the trade-offs. Also, although it works just fine, it hasn’t been tested thoroughly so it could break. Keep that in mind.
Working on this already (as agreed with @iMuzz).
Here’s an update on my progress:
static-tweets
ast parser is not finding the video media.Or, alternatively:
static-tweets
fetcher and parser to figure out how to fix it. Looks promising though. I’m only worried about CORS-type issues, but maybe there’s still a workaround by using iframes or similar. Will continue this afternoon.Will keep updating on this issue, and create a draft PR once I start having actual changes to push.