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

Facebook oEmbed Changes #161

Closed
ALazyMeme opened this issue Jun 9, 2021 · 1 comment · Fixed by #201
Closed

Facebook oEmbed Changes #161

ALazyMeme opened this issue Jun 9, 2021 · 1 comment · Fixed by #201

Comments

@ALazyMeme
Copy link
Contributor

ALazyMeme commented Jun 9, 2021

Refer this article: https://developers.facebook.com/docs/plugins/oembed/

The oEmbed product has been replaced with the oEmbed Read feature. If you implemented the oEmbed product before June 8, 2021, you have until September 7, 2021 to complete App Review for the oEmbed Read feature. If you have not been approved for the oEmbed Read feature by September 7, 2021, your oEmbed implementations will fail to load.

I don't really help with the api that often and am unsure if this actually affects us, but I thought I'd bring it to attention anyway.

@ALazyMeme ALazyMeme changed the title Facebook oEmbed Migration Facebook oEmbed Changes Jun 9, 2021
@KararTY
Copy link
Contributor

KararTY commented Jun 9, 2021

This does indeed affect existing API keys but doesn't affect the code.

However, an update to providers.json is necessary. Here's the relevant commit for updating providers.json:
iamcal/oembed@6f2af31

When it comes to fixing existing API keys, one needs to do the new App Review process:
image
Also, they have an entire video over here https://developers.facebook.com/docs/app-review describing the process.

Also, services must now undergo yearly data checkup https://developers.facebook.com/docs/development/maintaining-data-access/data-use-checkup It's not very difficult, just lots of boxes to check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants