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

Check if thumbnail_url is set in getPhotoHtml #13825

Merged
merged 1 commit into from
Feb 12, 2019
Merged

Check if thumbnail_url is set in getPhotoHtml #13825

merged 1 commit into from
Feb 12, 2019

Conversation

spacedmonkey
Copy link
Member

@spacedmonkey spacedmonkey commented Feb 11, 2019

Description

Thumbnail_url is an optional field in oembed endpoints. Check it is set first before using and fallback to url if not.

More details can be found in #13426

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

thumbnail_url is an optional field in oembed endpoints. Check it is set first before using and fallback to url if not.
@notnownikki notnownikki added this to the 5.1 (Gutenberg) milestone Feb 12, 2019
@notnownikki notnownikki added the [Block] Embed Affects the Embed Block label Feb 12, 2019
Copy link
Member

@notnownikki notnownikki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, this fixes giphy and makes photo handling work as it should.

@youknowriad youknowriad merged commit 6d6e3c4 into WordPress:master Feb 12, 2019
@youknowriad
Copy link
Contributor

Congrats for the first contribution @spacedmonkey

youknowriad pushed a commit that referenced this pull request Mar 6, 2019
thumbnail_url is an optional field in oembed endpoints. Check it is set first before using and fallback to url if not.
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
thumbnail_url is an optional field in oembed endpoints. Check it is set first before using and fallback to url if not.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Embed Affects the Embed Block
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants