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

Instagram multi-posts with videos #84

Closed
aaronpk opened this issue Jan 10, 2019 · 2 comments
Closed

Instagram multi-posts with videos #84

aaronpk opened this issue Jan 10, 2019 · 2 comments

Comments

@aaronpk
Copy link
Owner

aaronpk commented Jan 10, 2019

Currently Instagram posts with a mix of photos and videos are not handled well, the videos are just dropped. This is mostly due to the fact that there is no clear way to represent this in Microformats, since there is no way to express ordering between photo and video values.

Here is an example post with a mix of photos and videos: https://www.instagram.com/p/BZWmpecjBwN/

@swentel
Copy link
Contributor

swentel commented Apr 24, 2020

I've been thinking about this one the last few days, and I wonder whether it would make sense to store the canonical url of the post into the video property. We'd have to document that clients can't directly use a video player, but are responsible to fetch the URL themselves and parse it, optionally using a call back to the microsub server, as the 'preview' call probably shall return the actual video url.

That would solve it without the microsub server having to do a lot of round trips when importing an instagram feed, although, in the end, that might end up to be more performant.

@aaronpk
Copy link
Owner Author

aaronpk commented Jan 28, 2022

This is mostly irrelevant now that it's basically impossible to pull data from Instagram so I'm just going to close it.

@aaronpk aaronpk closed this as completed Jan 28, 2022
@aaronpk aaronpk removed the blocked label Jan 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants