-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
embedo dosen't work now, FB/IG seemd change the API #25
Comments
Please upgrade the version with the fixes |
Hi, would be very nice if you can make a new release to support the newest facebook and instagram api. Feel free to contact me if we can help. |
Looks like app id & access tokens are now required so will be higher barrier to entry. Would be happy to take a stab at PR but so far, I can't seem to get test suite to pass. Every case apart from embedding a video fails, seemingly because load event is never emitted 🤷 |
Thanks for reporting. I will look into it in evening, and update oembed URLs and specs. |
I've tested and release new minor version for facebook and instagram embeds, only thing which needs to be done is just add See #26 for screenshots and changes I've made. |
it seem that the demo site haven't be updated? |
hi I find fb update their API, so could you upgrade base on the new version?
FYI https://developers.facebook.com/docs/instagram/oembed-legacy/
thanks a lot
The text was updated successfully, but these errors were encountered: