Why does the API often fail to create thumbnails although big enough images are downloaded? Example, if you paste this in an opening post… [image] Legal experts are freaking out about Bill Barr’s actions to help Trump win -... "The attorney general is a threat to American citizens havi…

Yes, thanks @merefield , but we just use @david ’s theme component, not the TLP TC. But I will probably use your eval to patch core if we don’t get some movement or an option provided by the @team on this issue.

The above render is from the TLP TC, but yes, I use a back-end plugin (the theme ‘side car’) to modify the Ruby :). https://github.com/merefield/discourse-topic-previews/tree/theme_sidecar I appreciate that may not be an option if hosted.

[image] Terrapop: it’s currently not on the table [image] Terrapop: we don’t get some movement or an option provided by the @team This is totally on the table, that’s why it is tagged pr-welcome :smiley: Check this post for a starting point for implementation.

@merefield Kindly can you do the PR? Unfortunately, I am completely new to Ruby. Or how is the process for a PR?

@david I would definitely be delighted to be able to remove this override from the plugins and agree a core solution would be best.

Understood. I just have a simple docker dev environment for developing some simple plugins and core patches, yet. Will wait until we have someone to look at this and probably patch via your eval in the meantime.

A proper job here is not an immaterial project, because you need to modify test cases, and work across the two components (discourse and discourse one-box). But definitely worth doing! Ruby is nothing to fear after Ember btw :wink:

Yes, but this is defo out of my current scope and abilities unfortunately.

As a work around I often manually copy and paste the image into a whisper, then select the thumbnail from that. Automatic would be nicer, perhaps with a config item to give domains where thumbnails shouldn’t be taken from one boxes (i.e. to cover the github case).

Using onebox images for topic thumbnails

Development

merefield (Robert) October 9, 2020, 12:08pm 14

@Terrapop here are the Contributing to Discourse development

I would be a more immediate assistant on this but rammed with client work atm.

Topic		Replies	Views
Thumbnails are missing for our external embeds when using topic-thumbnails Development pr-welcome	10	1178	March 31, 2023
Problems with thumbnails from Soundcloud and YouTube oneboxes Support	11	2064	June 3, 2020
Oneboxing uses full size thumbnail images UX	10	1377	March 27, 2019
Issue with Onebox and Youtube Bug onebox	10	1322	August 20, 2019
Add Onebox integration; show thumbnail with topic-thumbnails plugin Development	1	621	March 31, 2023

Using onebox images for topic thumbnails

Related topics